Paths

Table of Contentst

Correctly identify `@todo` comments as TODOs.
ClosedPublic
Actions

Authored by joshuaspence on Mar 3 2014, 12:17 PM.

Tags

None

Referenced Files

	F21202429: D8389.diff
	Sun, Jul 5, 7:49 AM

	Unknown Object (File)
	Mon, Jun 29, 9:54 PM

	Unknown Object (File)
	Apr 16 2026, 9:29 PM

	Unknown Object (File)
	Mar 27 2026, 7:42 PM

	Unknown Object (File)
	Mar 8 2026, 7:09 PM

	Unknown Object (File)
	Feb 9 2026, 10:30 AM

	Unknown Object (File)
	Feb 7 2026, 11:04 AM

	Unknown Object (File)
	Jan 12 2026, 12:48 AM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARCfb826bbf9cf0: Correctly identify `@todo` comments as TODOs.

Summary

Allow @todo comments to be linted as TODOs as well as TODO comments.

Test Plan

I added a new test case (todo.lint-test)

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Seems reasonable to me.

This revision is now accepted and ready to land.Mar 4 2014, 7:03 PM

Closed by commit rARCfb826bbf9cf0 (authored by @joshuaspence, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistXHPASTLinter.php

8 lines

__tests__/

xhpast/

todo.lint-test

10 lines

Diff 19936

View Options

src/lint/linter/ArcanistXHPASTLinter.php

View Options

src/lint/linter/tests/xhpast/todo.lint-test

Correctly identify `@todo` comments as TODOs.ClosedPublicActions