Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Integrate phpcs-patch into Travis and pre-commit hook #77

Closed
westonruter opened this issue Jan 15, 2015 · 1 comment
Closed

Integrate phpcs-patch into Travis and pre-commit hook #77

westonruter opened this issue Jan 15, 2015 · 1 comment

Comments

@westonruter
Copy link
Contributor

We need to facilitate only reporting PHP_CodeSniffer issues for lines that are part of changes in the pull request (on Travis) or as part of the current commit (Travis or pre-commit hook).

This depends on xwp/PHP_CodeSniffer#5

When running on Travis, we need to look at the TRAVIS_COMMIT_RANGE environment variable. For pre-commit, we obviously just look at the stage.

The work here is needed for integrating PHP_CodeSniffer into the WordPress Core build tools (#30153).

@westonruter
Copy link
Contributor Author

I've worked out a way for Travis to filter PHP_CodeSniffer and JSHint issues just to issues reported on the changes within a PR: #95.

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

No branches or pull requests

1 participant