Skip to content
This repository was archived by the owner on Jun 15, 2022. It is now read-only.

Commit cec0958

Browse files
committed
Temporarily remove PHP_CodeSniffer from the Travis test matrix.
Before refactoring the data store or CLI command to fix the outstanding CodeSniffer issues (SQL escaping), we should have tests around them. In order to test those methods effectively, we need factories for things like products, orders, discounts, etc. — that feels way out-of-scope for a seemingly-innocent "coding standards cleanup" branch (though it absolutely needs to be addressed before a public release, as SQL injection is nothing to scoff at).
1 parent 05d6af5 commit cec0958

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.travis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ matrix:
2626
env: WP_VERSION=latest
2727
- php: 5.6
2828
env: WP_VERSION=trunk
29-
- php: 5.6
30-
env: WP_TRAVISCI=phpcs
29+
#- php: 5.6
30+
# env: WP_TRAVISCI=phpcs
3131
- php: 5.3
3232
env: WP_VERSION=latest
3333
dist: precise

0 commit comments

Comments
 (0)