Page MenuHomePhabricator

Add documentation about the script and regex linter to the user guide.
ClosedPublic

Authored by wotte on May 13 2014, 2:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 10, 7:01 AM
Unknown Object (File)
Sun, Dec 29, 8:49 PM
Unknown Object (File)
Thu, Dec 26, 10:14 AM
Unknown Object (File)
Dec 12 2024, 4:06 AM
Unknown Object (File)
Dec 9 2024, 3:02 AM
Unknown Object (File)
Dec 4 2024, 7:20 AM
Unknown Object (File)
Dec 2 2024, 5:26 AM
Unknown Object (File)
Dec 1 2024, 7:49 AM
Subscribers

Details

Summary

The big, gigantic comment about the script and regex linter belongs in a more obvious place. I think this is a more obvious place. I also cleaned up a couple things.

I'll update D9084 to remove the big comment block and point here instead.

Test Plan

bin/diviner generate --book src/docs/book/user.book

Diff Detail

Repository
rP Phabricator
Branch
snr_diviner
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/docs/user/userguide/arcanist_lint_script_and_regex.diviner:68TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 432
Build 432: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

wotte retitled this revision from to Add documentation about the script and regex linter to the user guide..
wotte updated this object.
wotte edited the test plan for this revision. (Show Details)
wotte added a reviewer: epriestley.
wotte edited edge metadata.
epriestley edited edge metadata.
epriestley added inline comments.
src/docs/user/userguide/arcanist_lint.diviner
416

"an linter"

This revision is now accepted and ready to land.May 13 2014, 8:50 PM
epriestley updated this revision to Diff 21628.

Closed by commit rP0ab192d24586 (authored by @wotte, committed by @epriestley).

src/docs/user/userguide/arcanist_lint.diviner
416

me fail english? that's unpossible!