Page MenuHomePhabricator
Differential D9802

Alphabetize `.arclint` linters
ClosedPublic
Actions

Authored by joshuaspence on Jul 2 2014, 10:00 PM.
Tags
None
Referenced Files
F13242356: D9802.diff
Thu, May 23, 2:38 AM
F13225602: D9802.id23529.diff
Sun, May 19, 4:12 PM
F13221162: D9802.diff
Sun, May 19, 2:19 AM
F13215614: D9802.id23522.diff
Fri, May 17, 6:34 PM
F13202211: D9802.diff
Tue, May 14, 10:23 PM
Unknown Object (File)
Wed, Apr 24, 10:45 PM
Unknown Object (File)
Apr 20 2024, 7:12 PM
Unknown Object (File)
Apr 18 2024, 3:26 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHU8549f7d2a559: Alphabetize `.arclint` linters
Summary

I'm a bit OCD with this, but I find the .arclint file easier to read if the keys are in alphabetical order.

Test Plan

N/A

Diff Detail

Repository
rPHU libphutil
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1490
Build 1490: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 23522.Jul 2 2014, 10:00 PM
joshuaspence retitled this revision from to Alphabetize `.arclint` linters.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 2 2014, 10:00 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1490: Diff 23522.Jul 2 2014, 10:02 PM
epriestley accepted this revision.Jul 3 2014, 1:50 AM
epriestley edited edge metadata.

Haha. Pretty fluff, but seems reasonable. I don't think linter order otherwise conveys any meaningful information.

This revision is now accepted and ready to land.Jul 3 2014, 1:50 AM
joshuaspence closed this revision.Jul 3 2014, 1:53 AM
joshuaspence updated this revision to Diff 23529.

Closed by commit rPHU8549f7d2a559 (authored by @joshuaspence).

Harbormaster completed remote builds in B1490: Diff 23522.Jul 9 2014, 7:01 PM

Revision Contents
Changeset List

PathSize
12 lines

Diff 23522

View Options

.arclint

Loading...
Phabricator · Built by Phacility