Skip to content

Ditch very old phpcs implementation in favor of WPCS#4057

Merged
schlessera merged 10 commits intomasterfrom
3940-phpcs
May 16, 2017
Merged

Ditch very old phpcs implementation in favor of WPCS#4057
schlessera merged 10 commits intomasterfrom
3940-phpcs

Conversation

@danielbachhuber
Copy link
Copy Markdown
Member

Fixes #3940

@danielbachhuber danielbachhuber added this to the 1.2.0 milestone May 9, 2017
@danielbachhuber danielbachhuber added the scope:testing Related to testing label May 9, 2017
.travis.yml Outdated
- ./ci/prepare.sh

script:
- phpcs --standard=phpcs.ruleset.xml $(find . -name '*.php')
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@schlessera Any ideas why this passes on Travis, when it's expected to fail?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nevermind — user error :)

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

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants