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 4273 Build 4286: [Placeholder Plan] Wait for 30 Seconds
Time | Test | |
---|---|---|
98 ms | ArcanistTextLinterTestCase::testLinter | |
726 ms | ArcanistCSSLintLinterTestCase::testLinter | |
110 ms | ArcanistCSSLintLinterTestCase::testVersion | |
121 ms | ArcanistChmodLinterTestCase::testLinter | |
245 ms | ArcanistClosureLinterTestCase::testLinter | |
View Full Test Results (1 Failed · 53 Passed · 6 Skipped) |
Event Timeline
src/lint/linter/ArcanistTextLinter.php | ||
---|---|---|
18 | This causes a behavioral change, right? That's maaaaybe OK but likely surprising for users... |
src/lint/linter/ArcanistTextLinter.php | ||
---|---|---|
18 | 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. |