Page MenuHomeDevCentral
Differential D938

Add linter name to lint message
ClosedPublic
Actions

Authored by dereckson on Mar 14 2017, 13:30.
Tags
None
Referenced Files
F24842180: D938.diff
Fri, Mar 13, 19:01
F24838537: D938.id2389.diff
Fri, Mar 13, 12:28
Unknown Object (File)
Thu, Mar 12, 09:49
Unknown Object (File)
Tue, Mar 10, 15:07
Unknown Object (File)
Sun, Mar 8, 19:03
Unknown Object (File)
Sat, Mar 7, 22:31
Unknown Object (File)
Fri, Mar 6, 19:26
Unknown Object (File)
Fri, Mar 6, 03:51
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rSCL7886c9031f5f: Add linter name to lint message
Summary

Set the default linter name as linter issue name.

See https://secure.phabricator.com/D11657#209615 for more context.

Test Plan

arc lint

Diff Detail

Repository
rSCL ShellCheck linter for Arcanist
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
linter
Build Status
Buildable 1453
Build 1701: arc lint + arc unit

Event Timeline

dereckson created this revision.Mar 14 2017, 13:30
Harbormaster completed remote builds in B1453: Diff 2389.Mar 14 2017, 13:31
dereckson accepted this revision.Mar 14 2017, 13:31
dereckson retitled this revision from Add linter name to Add linter name to lint message.
This revision is now accepted and ready to land.Mar 14 2017, 13:31
Closed by commit rSCL7886c9031f5f: Add linter name to lint message (authored by dereckson, committed by dereckson). · Explain WhyMar 14 2017, 13:31
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
lint/
linter/
1 line

Diff 2389

View Options

lint/linter/ArcanistShellCheckLinter.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator