Various tidying up of linting code.
Details
Details
- Reviewers
epriestley chad - Group Reviewers
Blessed Reviewers - Commits
- rARC67b6bed92ec2: Tidying up of linter code.
arc lint and arc unit still pass.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
The double-newline-between-methods is quasi-intentional, but I'll write a lint rule for it if we want to formalize it. I could also just stop doing it, it's definitely a little weird. Not sure if you have feelings one way or the other.
| src/lint/linter/ArcanistSpellingLinter.php | ||
|---|---|---|
| 145–154 | These should probably use the phutil_utf8_...() flavors, although I don't remember if we have phutil_utf8_ucwords(). | |
| src/lint/linter/__tests__/ArcanistLinterTestCase.php | ||
| 41–45 | This is probably wrong (or will soon be wrong) now, per discussion. | |
Comment Actions
Oh, based on what I had seen I figured the convention was for a single blank line between methods but a double blank line before some kind of header. I'm not fussed either way to be honest.