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
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7945
Build 8919: [Placeholder Plan] Wait for 30 Seconds
Build 8918: arc lint + arc unit

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 34034

View Options

src/workflow/ArcanistLintersWorkflow.php

Loading...
Phabricator · Built by Phacility