Page MenuHomePhabricator
Differential D11662

Remove the `ArcanistArcanistLinterTestCase`
ClosedPublic
Actions

Authored by joshuaspence on Feb 3 2015, 9:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 11:02 PM
Unknown Object (File)
Sat, Nov 16, 1:39 PM
Unknown Object (File)
Tue, Nov 12, 3:49 AM
Unknown Object (File)
Sat, Nov 9, 8:50 AM
Unknown Object (File)
Fri, Nov 8, 3:03 PM
Unknown Object (File)
Mon, Nov 4, 2:43 PM
Unknown Object (File)
Wed, Oct 23, 2:45 PM
Unknown Object (File)
Oct 18 2024, 7:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC992d939e3a5c: Remove the `ArcanistArcanistLinterTestCase`
Summary

I don't think that this provides too much value. I think that we should rework this to be inferred from the .arcconfig file perhaps?

Test Plan

arc unit

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 28069.Feb 3 2015, 9:20 PM
joshuaspence retitled this revision from to Remove the `ArcanistArcanistLinterTestCase`.
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 TranscriptFeb 3 2015, 9:20 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Feb 3 2015, 9:40 PM
epriestley edited edge metadata.

Yeah, these hardcodes feel really awkward. We can find some better place for this eventually.

This revision is now accepted and ready to land.Feb 3 2015, 9:40 PM
Closed by commit rARC992d939e3a5c: Remove the `ArcanistArcanistLinterTestCase` (authored by joshuaspence, committed by joshuaspence). · Explain WhyFeb 4 2015, 8:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 28100

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistArcanistLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistChmodLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistCommitLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistFilenameLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistGeneratedLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistJSONLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistMergeConflictLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistNoLintLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistPhutilXHPASTLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistSpellingLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistTextLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistXHPASTLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/ArcanistXMLLinterTestCase.php

Loading...
Phabricator · Built by Phacility