Skip to content

Conversation

@lucaboesch
Copy link
Contributor

This should fix all checks for Moodle coding style.

@lucaboesch lucaboesch force-pushed the moodlephpdoc branch 3 times, most recently from 52720cc to f1c8a3c Compare June 24, 2025 12:34
@dlmsr
Copy link
Member

dlmsr commented Jun 24, 2025

Thanks, I have removed the fix for the Behat tests for now. Partly
because the commit message says ‘Fix Moodle documentation errors.’
but mainly since I want to do some more refactoring work on the
feature files which will eventually have this fix included. 😉

@dlmsr dlmsr closed this Jun 24, 2025
@lucaboesch
Copy link
Contributor Author

Cool, all good.
In the Behat test it seemed that you had to be Admin to let the scheduled task run.

@lucaboesch
Copy link
Contributor Author

Whilst you're at it, #314 seems legit, too.

@dlmsr
Copy link
Member

dlmsr commented Jun 24, 2025 via email

@dlmsr
Copy link
Member

dlmsr commented Jun 24, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants