Page MenuHomePhabricator
Differential D13871

Add a linter rule for inline HTML
ClosedPublic
Actions

Authored by joshuaspence on Aug 11 2015, 2:31 PM.
Tags
None
Referenced Files
F15531679: D13871.id33527.diff
Wed, Apr 23, 12:12 PM
F15520661: D13871.id33527.diff
Sun, Apr 20, 7:10 AM
F15513004: D13871.id33498.diff
Thu, Apr 17, 3:34 PM
F15510510: D13871.id33498.diff
Wed, Apr 16, 8:16 PM
F15506800: D13871.diff
Tue, Apr 15, 2:18 PM
F15492193: D13871.id33527.diff
Sat, Apr 12, 12:12 PM
F15489521: D13871.id33498.diff
Fri, Apr 11, 9:34 AM
F15489135: D13871.id.diff
Fri, Apr 11, 6:18 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
Restricted Maniphest Task
T7409: Implement PHPCS lints
Commits
rARC2e76e2965cbe: Add a linter rule for inline HTML
Summary

Ref T7409. Based on InlineHTMLSniff.

Test Plan

Added unit tests.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 33498.Aug 11 2015, 2:31 PM
joshuaspence retitled this revision from to Add a linter rule for inline HTML.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence added a task: T7409: Implement PHPCS lints.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 11 2015, 2:31 PM
Herald added a subscriber: Korvin. · View Herald Transcript
epriestley accepted this revision.Aug 11 2015, 2:47 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 11 2015, 2:47 PM
• smij added a reviewer: • smij.Aug 11 2015, 8:34 PM
• smij removed a reviewer: • smij.Aug 11 2015, 8:37 PM
• smij added a task: Restricted Maniphest Task.
Closed by commit rARC2e76e2965cbe: Add a linter rule for inline HTML (authored by joshuaspence, committed by joshuaspence). · Explain WhyAug 13 2015, 9:42 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
lint/
linter/
31 lines
__tests__/
xhpast/
4 lines

Diff 33527

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/lint/linter/ArcanistInlineHTMLXHPASTLinterRule.php

Loading...
View Options

src/lint/linter/__tests__/xhpast/inline-html.lint-test

Loading...
Phabricator · Built by Phacility