Also check Filesystem::pathExists() in case the binary is local, for example <project root>/vendor/bin/phpcs
It might be worth mentioning that this was tested on windows.
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers
composer require --dev squizlabs/php_codesniffer, then edit composer.json and add
"config": { "bin-dir": "bin" },
Then in .arclint add:
"phpcs": { "type": "phpcs", "include": "(\\.php$|\\.inc$)", "bin": "vendor/bin/phpcs" }
and run arc lint.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- local-arc-lint-bin
- Lint
Lint Skipped - Unit
Test Failures - Build Status
Buildable 1008 Build 1008: [Placeholder Plan] Wait for 30 Seconds
Time | Test | |
---|---|---|
0 ms | testCSSLintLinter | |
0 ms | testPEP8Linter | |
0 ms | testPHPCSLint | |
0 ms | testPhutilXHPASTLint | |
0 ms | testRubyLint | |
View Full Test Results (8 Failed · 2 Passed · 11 Skipped) |
Event Timeline
Comment Actions
I thought I would take a stab at this, but I clearly did something wrong, as all the unit tests blew up on me.