← Back to team overview

yellow team mailing list archive

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