yellow team mailing list archive
-
yellow team
-
Mailing list archive
-
Message #01348
Re: Add yuidoc linter. (issue 6820047)
Looks good, Benji. Thanks again.
Gary
https://codereview.appspot.com/6820047/diff/7001/bin/lint-yuidoc
File bin/lint-yuidoc (right):
https://codereview.appspot.com/6820047/diff/7001/bin/lint-yuidoc#newcode13
bin/lint-yuidoc:13: '^ *(?:(\w+) *[:=] *function *|function +(\w+))
*\(')
Cool, thank you. looks good
https://codereview.appspot.com/6820047/diff/7001/bin/lint-yuidoc#newcode79
bin/lint-yuidoc:79: def remove(unwanted, seq):
that's a nicer function :-)
https://codereview.appspot.com/6820047/diff/7001/bin/lint-yuidoc#newcode118
bin/lint-yuidoc:118: print 'entries therein. The people of the future
thank you.'
Any message from the People of the Future (did you consider that
capitalization?) has my support.
https://codereview.appspot.com/6820047/
--
https://code.launchpad.net/~benji/juju-gui/doc-linter/+merge/131833
Your team Juju GUI Hackers is subscribed to branch lp:juju-gui.
References