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

Loosen thor dependency to ~> 0.18 #1740

Merged
merged 1 commit into from Aug 13, 2014

Conversation

Projects
None yet
3 participants
@postmodern
Contributor

postmodern commented Aug 12, 2014

Reduce the risk of dependendy conflicts with other libraries that pull in thor.

Loosen thor dependency to ~> 0.18
Reduce the risk of dependendy conflicts with other libraries that pull in thor.

@namusyaka namusyaka self-assigned this Aug 13, 2014

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Aug 13, 2014

Member

Seems reasonable, will pull this in (about to update changelog)

Member

nesquena commented Aug 13, 2014

Seems reasonable, will pull this in (about to update changelog)

nesquena added a commit that referenced this pull request Aug 13, 2014

Merge pull request #1740 from postmodern/patch-1
Loosen thor dependency to ~> 0.18

@nesquena nesquena merged commit 1bb7bf8 into padrino:master Aug 13, 2014

1 check failed

continuous-integration/travis-ci The Travis CI build failed
Details
@namusyaka

This comment has been minimized.

Show comment
Hide comment
@namusyaka

namusyaka Aug 13, 2014

Member

I'll send a patch about failing tests

Member

namusyaka commented Aug 13, 2014

I'll send a patch about failing tests

@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Aug 13, 2014

Member

Ah right, I saw that failure, sorry if I pulled this in prematurely. Will wait on your fix before I cut a release.

Member

nesquena commented Aug 13, 2014

Ah right, I saw that failure, sorry if I pulled this in prematurely. Will wait on your fix before I cut a release.

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