Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Refactor mixins #4343

Merged
merged 2 commits into from
Feb 23, 2017
Merged

Refactor mixins #4343

merged 2 commits into from
Feb 23, 2017

Conversation

chadwhitacre
Copy link
Contributor

Flat is better than nested.

@chadwhitacre
Copy link
Contributor Author

Ready for review @nobodxbodon @kaguillera @mattbk et al. I'd like to get this in master and then rebase #4305 to build on this.

@chadwhitacre
Copy link
Contributor Author

Redone to fix a test.

@kaguillera
Copy link
Contributor

LGTM

@mattbk
Copy link
Contributor

mattbk commented Feb 22, 2017

Don't wait for me.

@chadwhitacre
Copy link
Contributor Author

@kaguillera Merge? :-)

@chadwhitacre
Copy link
Contributor Author

Rebased, was 2d2b580.

@kaguillera kaguillera merged commit b3426f0 into master Feb 23, 2017
@chadwhitacre chadwhitacre mentioned this pull request Mar 3, 2017
@chadwhitacre chadwhitacre mentioned this pull request Apr 27, 2017
15 tasks
@chadwhitacre chadwhitacre deleted the refactor-mixins branch June 15, 2017 19:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants