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. | |