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
Enhancement: Update squizlabs/php_codesniffer and run phpcs before phpunit #542
Conversation
Wait, what? This shouldn't pass.
/cc @EvanDotPro |
Blocked by #543. |
318cf9f
to
22b7535
Compare
On Travis: There's not even any output generated. See https://travis-ci.org/ZF-Commons/ZfcUser/jobs/41857420#L175. |
5922964
to
127f078
Compare
@localheinz Yeah, i don't see any output(report) in travis at all. Something is going on with the command. |
127f078
to
ae8014a
Compare
There's not much of a difference to https://github.com/zendframework/modules.zendframework.com/blob/master/.travis.yml#L11, is there? |
@localheinz it seems to me that we reached green here: does that mean that we should merge? |
@@ -0,0 +1,17 @@ | |||
<?php |
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.
Should be removed
Not yet green, see this commit localheinz@ae8014a and the comments above. |
Closing since I've come to prefer |
This PR
squizlabs/php_codesniffer
phpcs.xml
for configuringphpcs
phpcs
beforephpunit
on TravisSomewhat related to zendframework/zendframework#6182.