Page MenuHomePhabricator
Differential D9100

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

Authored by wotte on May 13 2014, 2:16 PM.
Tags
None
Referenced Files
F13098147: D9100.id21628.diff
Fri, Apr 26, 4:48 AM
F13096816: D9100.id21612.diff
Thu, Apr 25, 7:02 PM
F13096815: D9100.id21628.diff
Thu, Apr 25, 7:02 PM
Unknown Object (File)
Thu, Apr 25, 1:27 AM
Unknown Object (File)
Wed, Apr 24, 10:44 PM
Unknown Object (File)
Wed, Apr 24, 3:08 PM
Unknown Object (File)
Sat, Apr 20, 11:02 AM
Unknown Object (File)
Fri, Apr 19, 3:27 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP0ab192d24586: Add documentation about the script and regex linter to the user guide.
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
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

wotte updated this revision to Diff 21612.May 13 2014, 2:16 PM
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.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 13 2014, 2:16 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
wotte edited the test plan for this revision. (Show Details)May 13 2014, 2:17 PM
wotte edited edge metadata.
wotte added a parent revision: D9084: Modernize the script and regex linter.
Harbormaster completed remote builds in B432: Diff 21612.May 13 2014, 2:18 PM
epriestley accepted this revision.May 13 2014, 8:50 PM
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 closed this revision.May 13 2014, 8:51 PM
epriestley updated this revision to Diff 21628.

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

wotte added inline comments.May 13 2014, 8:52 PM
src/docs/user/userguide/arcanist_lint.diviner
416

me fail english? that's unpossible!

Revision Contents
Changeset List

PathSize
src/
docs/
user/
userguide/
2 lines
153 lines

Diff 21628

View Options

src/docs/user/userguide/arcanist_lint.diviner

Loading...
View Options

src/docs/user/userguide/arcanist_lint_script_and_regex.diviner

Loading...
Phabricator · Built by Phacility