Skip to content

Test helper functions#74

Merged
radix merged 5 commits into
masterfrom
test-goodies
Dec 28, 2016
Merged

Test helper functions#74
radix merged 5 commits into
masterfrom
test-goodies

Conversation

@manishtomar
Copy link
Copy Markdown
Contributor

Some test related helper functions that we've been using in https://github.com/rackerlabs/otter/ for a long time.

@manishtomar
Copy link
Copy Markdown
Contributor Author

@radix is it ok to drop 3.6 support? flake8 and sphinx not working.

@radix
Copy link
Copy Markdown
Contributor

radix commented Dec 23, 2016

@manishtomar took me a bit to realize that you meant 2.6. I was like, wow, have I been gone from the Python world for that long!?

Anyway, given that Twisted no longer supports Python 2.6, I'm happy to have Effect drop support for it too. Go ahead and update the travis config to not run Python 2.6.

@radix
Copy link
Copy Markdown
Contributor

radix commented Dec 23, 2016

... and the README too, please :)

@manishtomar manishtomar mentioned this pull request Dec 27, 2016
@manishtomar
Copy link
Copy Markdown
Contributor Author

@radix What are your thoughts on adding nested_sequence function here?

@radix
Copy link
Copy Markdown
Contributor

radix commented Dec 27, 2016

@manishtomar That's fine. I'm not super happy with the function (the attrgetter part) but I can't really figure out anything better.

@manishtomar
Copy link
Copy Markdown
Contributor Author

@radix review please.

Copy link
Copy Markdown
Contributor

@radix radix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for the minor doc improvements too. :)

@radix radix merged commit ff86555 into master Dec 28, 2016
@manishtomar manishtomar deleted the test-goodies branch December 29, 2016 19:41
@manishtomar
Copy link
Copy Markdown
Contributor Author

@radix can you make a release? And maybe version it 0.11 since it has enhancements not a regression fix?

@radix
Copy link
Copy Markdown
Contributor

radix commented Jan 2, 2017

@manishtomar
Copy link
Copy Markdown
Contributor Author

@radix Thanks for the upload! And please don't say thank you! This is still one of my fav packages and am constantly looking forward to using it in all new projects. Thank you so much for writing this :) I'll try to find time to make further contributions.

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