Page MenuHomePhabricator

Modernize `ArcanistMergeConflictLinter`.
ClosedPublic

Authored by joshuaspence on May 10 2014, 7:41 AM.
Tags
None
Referenced Files
F14117159: D9039.id21464.diff
Thu, Nov 28, 5:35 PM
F14117158: D9039.id21471.diff
Thu, Nov 28, 5:35 PM
F14117157: D9039.id.diff
Thu, Nov 28, 5:35 PM
F14117037: D9039.diff
Thu, Nov 28, 5:14 PM
F14116900: D9039.diff
Thu, Nov 28, 4:54 PM
Unknown Object (File)
Mon, Nov 25, 9:59 AM
Unknown Object (File)
Thu, Nov 21, 10:53 AM
Unknown Object (File)
Sun, Nov 17, 1:32 AM
Subscribers

Details

Summary

The only real change here is adding a getLinterConfigurationName method so that this linter can be used with an .arclint file. Everything else is just some minor tidying.

Test Plan

N/A

Diff Detail

Repository
rARC Arcanist
Branch
merge-conflict
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 342
Build 342: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Modernize `ArcanistMergeConflictLinter`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 10 2014, 8:55 AM
epriestley updated this revision to Diff 21471.

Closed by commit rARCedd85a0e9db4 (authored by @joshuaspence, committed by @epriestley).