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

Make Listen::Listener capable of listening to multiple directories #95

Merged
merged 1 commit into from
Apr 8, 2013

Conversation

rymai
Copy link
Member

@rymai rymai commented Mar 29, 2013

Hi,

I noticed that the logic in Listen::MultiListener had nothing very specific and could be generalized directly in Listen::Listener.

It also deprecates Listen::MultiListener since all the logic has been
generalized and moved to Listen::Listener.

What do you think of this?

@thibaudgg
Copy link
Member

Having just Listen::Listener seems simpler, good for me. @Maher4Ever ok for you?

PS: Stats seems to be broken... :)

@rymai
Copy link
Member Author

rymai commented Apr 3, 2013

Thanks, you probably meant specs are broken! ;-)

@Maher4Ever any opinion about this PR?

@thibaudgg
Copy link
Member

@rymai please rebase on master and we should be good to go with 0.8.0 :)
@Maher4Ever are you ok with this PR?

It also deprecates Listen::MultiListener since all the logic has been
generalized and moved to Listen::Listener.
@rymai
Copy link
Member Author

rymai commented Apr 6, 2013

Done!

@coveralls
Copy link

Coverage increased (+0.06%) when pulling e11be0f on multi_listener_generalization into 81030fd on master.

View Details

@ghost ghost assigned Maher4Ever Apr 7, 2013
thibaudgg added a commit that referenced this pull request Apr 8, 2013
Make Listen::Listener capable of listening to multiple directories
@thibaudgg thibaudgg merged commit 386c0fd into master Apr 8, 2013
@rymai rymai deleted the multi_listener_generalization branch April 8, 2013 20:50
@rymai
Copy link
Member Author

rymai commented Apr 8, 2013

Thank you!

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.

4 participants