tpl: Add humanize func and cleanup lint #1818

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@moorereason
Member

moorereason commented Feb 5, 2016

Add humanize (inflect.Humanize) to the template funcMap. Documentation and
tests are included.

Various code cleanups of the template funcs:

  • Break pluralize and singularize out into stand-alone funcs.
  • Sort the list of funcMap entries.
  • Add some minimal godoc comments to all public funcs.
  • Fix some issues found by golint and grind.
tpl: Add humanize func and cleanup lint
Add humanize (inflect.Humanize) to the template funcMap.  Documentation and
tests are included.

Various code cleanups of the template funcs:

- Break pluralize and singularize out into stand-alone funcs.
- Sort the list of funcMap entries.
- Add some minimal godoc comments to all public funcs.
- Fix some issues found by golint and grind.
@bep

This comment has been minimized.

Show comment
Hide comment
@bep

bep Feb 6, 2016

Member

This looks good ... but it is two unrelated commits ... in one.

Member

bep commented Feb 6, 2016

This looks good ... but it is two unrelated commits ... in one.

@bep

This comment has been minimized.

Show comment
Hide comment
@bep

bep Feb 6, 2016

Member

Merged in 45df459

Member

bep commented Feb 6, 2016

Merged in 45df459

@bep bep closed this Feb 6, 2016

@moorereason moorereason deleted the moorereason:humanize branch Mar 16, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment