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
Unknown Object (File)
Thu, Sep 12, 8:05 AM
Unknown Object (File)
Thu, Sep 5, 4:09 PM
Unknown Object (File)
Aug 31 2024, 1:20 AM
Unknown Object (File)
Aug 30 2024, 5:05 PM
Unknown Object (File)
Aug 26 2024, 11:06 AM
Unknown Object (File)
Aug 25 2024, 6:24 AM
Unknown Object (File)
Aug 21 2024, 8:44 AM
Unknown Object (File)
Aug 17 2024, 9:00 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