Page MenuHomePhabricator
Differential D14084

arc linters: switch `name` and `short`
ClosedPublic
Actions

Authored by avivey on Sep 8 2015, 5:17 PM.
Tags
None
Referenced Files
F15290331: D14084.id34035.diff
Tue, Mar 4, 11:08 PM
Unknown Object (File)
Sun, Mar 2, 1:39 PM
Unknown Object (File)
Fri, Feb 21, 1:17 AM
Unknown Object (File)
Tue, Feb 11, 5:55 PM
Unknown Object (File)
Tue, Feb 11, 5:32 PM
Unknown Object (File)
Tue, Feb 11, 5:31 PM
Unknown Object (File)
Tue, Feb 11, 5:28 PM
Unknown Object (File)
Tue, Feb 11, 5:28 PM
View All Files
Subscribers
epriestley
Lint

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARCbdab422440f5: arc linters: switch `name` and `short`
Summary

I think they were wrong.

Test Plan

arc linters nolint. It would fail without it.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

avivey updated this revision to Diff 34034.Sep 8 2015, 5:17 PM
avivey retitled this revision from to arc linters: switch `name` and `short`.
avivey updated this object.
avivey edited the test plan for this revision. (Show Details)
avivey added a subscriber: Lint.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 8 2015, 5:17 PM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.Sep 8 2015, 5:39 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Sep 8 2015, 5:39 PM
Closed by commit rARCbdab422440f5: arc linters: switch `name` and `short` (authored by avivey, committed by epriestley). · Explain WhySep 8 2015, 5:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
workflow/
8 lines

Diff 34035

View Options

src/workflow/ArcanistLintersWorkflow.php

Loading...
Phabricator · Built by Phacility