Differential D19561

Consolidate redundant "should should" from some linter help strings in Arcanist
ClosedPublic
Actions

Authored by epriestley on Aug 3 2018, 7:59 PM.

Tags

None

Subscribers

None

Tokens

"Pterodactyl" token, awarded by epriestley.

Details

Reviewers

Commits

rARC9dfa68d739ee: (stable) Promote 2018 Week 31
rARCd9a4293ae734: Consolidate redundant "should should" from some linter help strings in Arcanist

Summary

See https://phabricator.wikimedia.org/T201138.

Test Plan

Read carefully.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 3 2018, 7:59 PM

Harbormaster completed remote builds in B20564: Diff 46765.Aug 3 2018, 7:59 PM

epriestley requested review of this revision.Aug 3 2018, 7:59 PM

should.jpg (720×960 px, 36 KB)

This revision is now accepted and ready to land.Aug 3 2018, 8:35 PM

epriestley awarded a token.Aug 3 2018, 8:40 PM

Closed by commit rARCd9a4293ae734: Consolidate redundant "should should" from some linter help strings in Arcanist (authored by epriestley). · Explain WhyAug 3 2018, 9:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

xhpast/

rules/

ArcanistDeprecationXHPASTLinterRule.php

2 lines

ArcanistUnsafeDynamicStringXHPASTLinterRule.php

6 lines

Diff 46771

src/lint/linter/xhpast/rules/ArcanistDeprecationXHPASTLinterRule.php

Loading...

src/lint/linter/xhpast/rules/ArcanistUnsafeDynamicStringXHPASTLinterRule.php

Loading...