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