Page MenuHomePhabricator

Modernize `ArcanistMergeConflictLinter`.
ClosedPublic

Authored by joshuaspence on May 10 2014, 7:41 AM.
Tags
None
Referenced Files
F15414220: D9039.id21464.diff
Wed, Mar 19, 11:30 PM
F15411398: D9039.id21471.diff
Wed, Mar 19, 9:16 AM
F15411397: D9039.id21464.diff
Wed, Mar 19, 9:16 AM
F15398938: D9039.id.diff
Mon, Mar 17, 1:56 AM
F15397990: D9039.diff
Sun, Mar 16, 11:08 PM
F15393381: D9039.id21471.diff
Sat, Mar 15, 8:49 PM
F15393380: D9039.id21464.diff
Sat, Mar 15, 8:49 PM
F15387713: D9039.id21464.diff
Sat, Mar 15, 2:48 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).