Modernize `ArcanistMergeConflictLinter`.
ClosedPublic
Actions

Authored by joshuaspence on May 10 2014, 7:41 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARCedd85a0e9db4: Modernize `ArcanistMergeConflictLinter`.

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 updated this revision to Diff 21464.May 10 2014, 7:41 AM

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.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 10 2014, 7:41 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B342: Diff 21464.May 10 2014, 7:42 AM

epriestley accepted this revision.May 10 2014, 8:55 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 10 2014, 8:55 AM

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

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistMergeConflictLinter.php

18 lines

Diff 21464

View Options

src/lint/linter/ArcanistMergeConflictLinter.php

Modernize `ArcanistMergeConflictLinter`.ClosedPublicActions