Skip to content

Conversation

OskarStark
Copy link
Contributor

added explanation, why it is no longer necessary

added explanation, why it is no longer necessary
@wouterj
Copy link
Member

wouterj commented Jun 25, 2015

👍 Ready imo

@OskarStark
Copy link
Contributor Author

ping

@javiereguiluz @xabbuh @stof

@javiereguiluz
Copy link
Member

👍 it looks OK to me.

@OskarStark thanks for this. I'm sure that this will be merged soon. Stay tuned!

@wouterj
Copy link
Member

wouterj commented Jun 26, 2015

Another try, hopefully I'll merge the correct PR now. I've reverted the previous wrong merge. Thanks again!

wouterj added a commit that referenced this pull request Jun 26, 2015
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5432).

Discussion
----------

removed squashing stuff. fixes #5368

added explanation, why it is no longer necessary

Commits
-------

e446316 removed squashing stuff
@wouterj wouterj closed this Jun 26, 2015
@OskarStark
Copy link
Contributor Author

great 😄

@OskarStark OskarStark deleted the patch-3 branch June 30, 2015 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants