Page MenuHomePhabricator
Differential D12474

Provide a getProjectRoot method for ArcanistLinter
ClosedPublic
Actions

Authored by joshuaspence on Apr 20 2015, 9:19 AM.
Tags
None
Referenced Files
F15529421: D12474.id30633.diff
Tue, Apr 22, 7:38 PM
F15507425: D12474.diff
Tue, Apr 15, 7:20 PM
F15504178: D12474.id30633.diff
Mon, Apr 14, 4:30 PM
F15479240: D12474.id30633.diff
Tue, Apr 8, 5:35 AM
F15446369: D12474.id31091.diff
Thu, Mar 27, 6:44 PM
F15436951: D12474.id30633.diff
Mar 25 2025, 4:50 PM
F15435595: D12474.diff
Mar 25 2025, 9:08 AM
F15431289: D12474.diff
Mar 24 2025, 11:36 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC1d72cfc94421: Provide a getProjectRoot method for ArcanistLinter
Summary

Adds a getProjectRoot() method to the ArcanistLinter class, simplifying a bunch of code.

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 29932.Apr 20 2015, 9:19 AM
joshuaspence retitled this revision from to Provide a getProjectRoot method for ArcanistLinter.
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 TranscriptApr 20 2015, 9:19 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
joshuaspence updated this revision to Diff 30633.May 7 2015, 12:56 PM
joshuaspence edited edge metadata.

Rebase

joshuaspence updated this revision to Diff 31091.May 19 2015, 12:56 PM

Rebase

epriestley accepted this revision.May 19 2015, 3:13 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/lint/linter/ArcanistLinter.php
254

state exception

This revision is now accepted and ready to land.May 19 2015, 3:13 PM
Closed by commit rARC1d72cfc94421: Provide a getProjectRoot method for ArcanistLinter (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 19 2015, 11:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 31164

View Options

src/lint/linter/ArcanistExternalLinter.php

Loading...
View Options

src/lint/linter/ArcanistLinter.php

Loading...
View Options

src/lint/linter/ArcanistScriptAndRegexLinter.php

Loading...
View Options

src/lint/linter/ArcanistSpellingLinter.php

Loading...
Phabricator · Built by Phacility