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
Feature PHPDoc check #85
Feature PHPDoc check #85
Conversation
9285a32
to
522f9c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The build is failing due to code formatting I think. Please run make test
and it'll take care of it for you (commit the result).
Instead of directory, use MoodlePlugin::getFiles call to identify the list of files required for testing and pass them to moodlecheck cli script.
Hello @polothy |
Heh, previous build apparently failed on phpdoc check 😃(https://travis-ci.org/blackboard-open-source/moodle-plugin-ci/jobs/481097455#L728) Fixed in b58488f. |
This feature adds phpdoc check using moodle-local_moodlecheck plugin (closing #10).
This command does not include command unittest though, as opposed to other commands, in order to be able to run phpdoc check as a part of unittest, a full Moodle installation is required (local_moodlecheck is using core moodle functions). If unittest is required, I guess I need to add an alternative to MoodleTestCase that would included cloning process instead of using fixture and use it for this command test.