Page MenuHomePhabricator

Move the "ragged classtree edge" linter rule to `ArcanistPhutilXHPASTLinter`.
ClosedPublic

Authored by joshuaspence on Jun 17 2014, 2:37 AM.
Tags
None
Referenced Files
F18805923: D9590.id22996.diff
Sat, Oct 18, 5:00 PM
F18802957: D9590.id22982.diff
Fri, Oct 17, 9:03 PM
F18772978: D9590.diff
Thu, Oct 9, 6:09 AM
F18764810: D9590.id22996.diff
Tue, Oct 7, 8:00 AM
F18752602: D9590.id22982.diff
Sat, Oct 4, 3:51 PM
F18719871: D9590.id22996.diff
Sep 29 2025, 6:02 PM
F18709838: D9590.id.diff
Sep 29 2025, 1:02 AM
F18672573: D9590.diff
Sep 25 2025, 7:48 AM
Subscribers

Details

Summary

This linter rule is very specific to the Phabricator and it is unlikely that it is being used elsewhere. We should move it to ArcanistPhutilXHPASTLinter, where other Phabricator-specific linting code lives.

Test Plan

Moved the corresponding test case.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Move the "ragged classtree edge" linter rule to `ArcanistPhutilXHPASTLinter`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 17 2014, 11:10 AM