Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moodle Plugin CI, and Moodle coding style #728

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from

Conversation

lucaboesch
Copy link
Contributor

@lucaboesch lucaboesch commented Jul 26, 2024

In these commits, I wanted to prove that the plagiarism_turnitin plugin could well be rewritten/reformatted to adhere to Moodle coding guidelines.
The submitted did not completely fix PHPUnit and Behat tests, though.

Thanks for taking into consideration. If desired, I can also divide into smaller pieces, of course.

@jack-tii
Copy link
Contributor

jack-tii commented Jul 29, 2024

Thanks very much for this PR, we should definitely be following the Moodle style guides.
We are currently having internal discussions about how best to add Github workflows to our plugins as described in https://moodlehq.github.io/moodle-plugin-ci/
Merging this would therefore have to wait until after our next TT release, which is currently planned for August 1st. I will keep you updated regarding our decisions around this, as this is a feature I would love to have.

@lucaboesch
Copy link
Contributor Author

I can understand very well. Please refer to #731 where the same code lies but with the omission of Github workflows.

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

Successfully merging this pull request may close these issues.

4 participants