Details
Details
- Reviewers
epriestley chad - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T5105: Ensure that `ArcanistTextLinter` respects `.editorconfig` settings
- Commits
- rARCf2a3fdf5e393: Fix text lint issues
arc lint --everything
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- master
- Lint
Lint Passed - Unit
Test Failures - Build Status
Buildable 4452 Build 4466: [Placeholder Plan] Wait for 30 Seconds
Time | Test | |
---|---|---|
100 ms | ArcanistTextLinterTestCase::testLinter | |
68 ms | ArcanistBaseCommitParserTestCase::testBasics | |
63 ms | ArcanistBaseCommitParserTestCase::testHalt | |
64 ms | ArcanistBaseCommitParserTestCase::testJump | |
64 ms | ArcanistBaseCommitParserTestCase::testJumpReturn | |
View Full Test Results (1 Failed · 69 Passed · 6 Skipped) |
Event Timeline
src/lint/linter/ArcanistTextLinter.php | ||
---|---|---|
18 ↗ | (On Diff #28017) | This causes a behavioral change, right? That's maaaaybe OK but likely surprising for users... |
src/lint/linter/ArcanistTextLinter.php | ||
---|---|---|
18 ↗ | (On Diff #28017) | Yes it does, but it seemed necessary to make this work. Otherwise, we can't use max_line_length = in an .editorconfig file to mean "no maximum line length". We could maybe tweak this to use some non-null sentinel value I suppose. |