The 'hash' key in composer.json files was removed by composer version 1.3.0 in December 2016, in favour of the 'content-hash' key.
Update the code to validate the content-hash instead.
Fixes T15647
Details
Details
- Reviewers
avivey - Group Reviewers
O1: Blessed Committers - Maniphest Tasks
- T15647: 'arc lint' fails for composer.json with RuntimeException: Undefined array key "hash"
- Commits
- rARCba42b63704b2: Update ArcanistComposerLinter.php to check content-hash instead of hash
- Update a composer.json file, without running 'composer update'
- Run 'arc lint' and confirm it warns you that the composer.lock file is out of date
- Run 'composer update'
- Run 'arc lint' and confirm it returns OKAY
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- composerlinthash (branched from master)
- Lint
Lint Passed - Unit
Test Failures - Build Status
Buildable 852 Build 852: arc lint + arc unit
| Time | Test | |
|---|---|---|
| 270 ms | ArcanistPhpcsLinterTestCase::testLinter Lint emitted an unexpected set of messages for file "basics.lint-test".
MISSING MESSAGES
Message with severity "error" at "2:1"
| |
| 12 ms | ArcanistChmodLinterTestCase::testLinter 5 assertion(s) passed. | |
| 0 ms | ArcanistClosureLinterTestCase::testVersion 1 assertion(s) passed. | |
| 0 ms | ArcanistCpplintLinterTestCase::testVersion 1 assertion(s) passed. | |
| 11 ms | ArcanistFilenameLinterTestCase::testLinter 5 assertion(s) passed. | |
| View Full Test Results (1 Failed · 23 Passed · 29 Skipped) |