Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[abjonnes-sorted_modules] Sorting modules #169

Merged
merged 1 commit into from Apr 5, 2015
Merged

Conversation

kaneda
Copy link
Contributor

@kaneda kaneda commented Apr 5, 2015

- print the registered modules in sorted order, then print the patterns and corresponding functions in a more easy-to-read and sorted fashion
- Updated from #164 and #168
@kaneda
Copy link
Contributor Author

kaneda commented Apr 5, 2015

@abjonnes for future reference:

  • always checkout against the latest master from your upstream before beginning work, then fetch upstream and rebase against upstream before asking to merge down.
  • we use 4 spaces for tabs, not 3
  • we don't use tabs at all(see above)
  • our max line-length is 120. the guidance is that if it fits in a github diff it's fine, otherwise try to shorten

kaneda added a commit that referenced this pull request Apr 5, 2015
[abjonnes-sorted_modules] Sorting modules
@kaneda kaneda merged commit 63a47a1 into master Apr 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant