Separate method selection from including/extending #5

Merged
merged 1 commit into from May 6, 2012

Conversation

Projects
None yet
2 participants
Collaborator

misfo commented May 6, 2012

This make it clear that methods are simply being included or or extending the class. Also, this avoids having to include Rns.

/cc @muness. About your comment, I do agree that this is partly a stylistic change. I guess the reason I prefer this is that include and extend already do what we want them to do (include some methods or extend the class with some methods), so this keeps the orthogonal concern of grouping the right methods together separate. Thoughts?

@misfo misfo Separate method selection from including/extending
This make it clear that methods are simply being included or or
extending the class. Also, this avoids having to include Rns.
31e9935

@alandipert alandipert pushed a commit that referenced this pull request May 6, 2012

Alan Dipert Merge pull request #5 from misfo/module_with
Separate method selection from including/extending
8b7c94b

@alandipert alandipert merged commit 8b7c94b into alandipert:master May 6, 2012

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