Page MenuHomePhabricator
Differential D13896

Improve the "inline HTML" linter rule
ClosedPublic
Actions

Authored by joshuaspence on Aug 14 2015, 4:43 AM.
Tags
None
Referenced Files
F19530366: D13896.id33639.diff
Mon, Jan 19, 10:16 PM
F19521174: D13896.diff
Fri, Jan 16, 8:26 PM
F19509268: D13896.id33639.diff
Jan 10 2026, 5:05 PM
F19448169: D13896.id33639.diff
Dec 29 2025, 5:31 PM
F19317540: D13896.id.diff
Dec 25 2025, 5:49 AM
F19304722: D13896.diff
Dec 24 2025, 3:24 PM
F19082595: D13896.id33639.diff
Dec 2 2025, 11:46 AM
F18825227: D13896.diff
Oct 23 2025, 8:26 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC4404e66735c4: Improve the "inline HTML" linter rule
Summary

Improve ArcanistInlineHTMLXHPASTLinterRule such that it doesn't raise duplicate warnings. Also be a bit more lax with whitespace.

Test Plan

Unit tests now pass.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 33544.Aug 14 2015, 4:43 AM
joshuaspence retitled this revision from to Improve the "inline HTML" linter rule.
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 TranscriptAug 14 2015, 4:43 AM
Herald added a subscriber: Korvin. · View Herald Transcript
epriestley accepted this revision.Aug 14 2015, 10:58 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 14 2015, 10:58 AM
Closed by commit rARC4404e66735c4: Improve the "inline HTML" linter rule (authored by joshuaspence, committed by joshuaspence). · Explain WhyAug 19 2015, 5:34 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
5 lines
__tests__/
xhpast/
1 line
1 line

Diff 33639

View Options

src/lint/linter/ArcanistInlineHTMLXHPASTLinterRule.php

Loading...
View Options

src/lint/linter/__tests__/xhpast/embedded-tags.lint-test

Loading...
View Options

src/lint/linter/__tests__/xhpast/php-tags-bad.lint-test

Loading...
Phabricator · Built by Phacility