Add linter rule for invalid modifiers
ClosedPublic
Actions

Authored by joshuaspence on May 19 2015, 11:13 AM.

Tags

None

Referenced Files

	F15516975: D12922.diff
	Fri, Apr 18, 10:43 PM

	F15516083: D12922.id.diff
	Fri, Apr 18, 3:59 PM

	F15512905: D12922.diff
	Thu, Apr 17, 2:47 PM

	F15503600: D12922.id31171.diff
	Mon, Apr 14, 11:12 AM

	F15479981: D12922.id31170.diff
	Tue, Apr 8, 10:37 AM

	F15470783: D12922.diff
	Sat, Apr 5, 1:12 AM

	F15464345: D12922.id31171.diff
	Wed, Apr 2, 8:41 AM

	F15447752: D12922.id31170.diff
	Fri, Mar 28, 1:22 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARC097c894d0831: Add linter rule for invalid modifiers

Summary

Add some linter rule to detect invalid modifiers for class methods/properties.

Test Plan

Added test cases.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 31088.May 19 2015, 11:13 AM

joshuaspence retitled this revision from to Add linter rule for invalid modifiers.

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 19 2015, 11:13 AM

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

epriestley accepted this revision.May 19 2015, 12:59 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 19 2015, 12:59 PM

Rebase

Closed by commit rARC097c894d0831: Add linter rule for invalid modifiers (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 19 2015, 11:47 PM

This revision was automatically updated to reflect the committed changes.

joshuaspence mentioned this in T6154: XHPAST fails to recognise some syntax errors.May 27 2015, 9:28 AM

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistXHPASTLinter.php

113 lines

__tests__/

xhpast/

invalid-modifiers.lint-test

18 lines

Diff 31171

View Options

src/lint/linter/ArcanistXHPASTLinter.php

View Options

src/lint/linter/tests/xhpast/invalid-modifiers.lint-test

Add linter rule for invalid modifiersClosedPublicActions