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

Another attempt to resolve six=1.10.0 version conflict. #21

Merged
merged 5 commits into from
Jun 19, 2018

Conversation

fulv
Copy link
Member

@fulv fulv commented Jun 19, 2018

No description provided.

@ida
Copy link

ida commented Jun 19, 2018

That looks good! I'll add that to the other PR, to see if tests will pass, then.

@ida
Copy link

ida commented Jun 19, 2018

I see you've dine that already but's still breaking. That's funny, if there's no difference between the branches but a changed README. Now looking if someone else has the same error...
Thanks for continuing to find a solution! (@fulv)

@fulv
Copy link
Member Author

fulv commented Jun 19, 2018

@ida I have no idea why, but it seems like travis was always restarting the tests from your first commit, instead of the latest. As a consequence, we don't know whether any of the intermediate commits actually fixed the problem or not.
Anyway, merging now.

@fulv fulv closed this Jun 19, 2018
@fulv fulv reopened this Jun 19, 2018
@fulv fulv merged commit c8eed75 into master Jun 19, 2018
@fulv fulv deleted the fulv-regressionInfo branch June 19, 2018 14:01
@ida
Copy link

ida commented Aug 29, 2018

@fulv Lost this out of sight, if you find some time for a release, that'll be süper.

@fulv
Copy link
Member Author

fulv commented Aug 29, 2018 via email

@ida
Copy link

ida commented Aug 30, 2018

Oh cool, thanks. Have a good time!

@ida
Copy link

ida commented Sep 26, 2018

@fulv If you don't find time, can you please regain ownership to my user ('ida'), so I can do it myself? Thanks!

@ida
Copy link

ida commented Oct 11, 2018

@fulv Can't find our latest convo, but I see now you've been so kind to change the README and make a release, huuuuge thanks for that! And have a great day!

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.

2 participants