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

Too many coding style violations #7

Open
mudrd8mz opened this issue Jun 3, 2019 · 5 comments
Open

Too many coding style violations #7

mudrd8mz opened this issue Jun 3, 2019 · 5 comments
Labels

Comments

@mudrd8mz
Copy link

mudrd8mz commented Jun 3, 2019

The Moodle coding style prechecker results https://moodle.org/plugins/pluginversion.php?id=19703&smurf=html report hundreds on formal coding style violations, many of them quite trivial to fix. It helps a lot with reviewing, debugging and contributing when plugins follow the common Moodle coding guidelines. Please consider fixing these.

@nexterday
Copy link
Contributor

Thats true - we wanted to reserve the name in moodle plugins directory, and then cleanup (and most likely modify the plugin) to fully comply with moodle. we have many other plugins with the same case, but we have limited time for them at the moment and on the assumption that approval takes a while, we would have made the required changes by the time the approval process starts

@mudrd8mz
Copy link
Author

mudrd8mz commented Jun 3, 2019

Ah, right. Thanks for clarification. Shall we mark all the other plugins as needing more work too? To give your team a bit more time to deal with reported coding style issues.

@nexterday
Copy link
Contributor

Yes please. we also have a major plugin coming this year (LTI qype) and this one will be submitted in full compliance (apart from external lib such as oAuth).

@mudrd8mz
Copy link
Author

mudrd8mz commented Jun 3, 2019

Ok then, no worries and thanks for the heads up notice. Please ping us whenever you're ready with the updated versions.

@nexterday nexterday added the TODO label Jun 3, 2019
@mudrd8mz
Copy link
Author

Just noting that a new version has been submitted for approval review in 2 March - https://tracker.moodle.org/browse/CONTRIB-7668

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

No branches or pull requests

2 participants