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

build: Add 3.6 to build matrix #37

Merged
merged 1 commit into from Aug 12, 2020

Conversation

@EvanPurkhiser
Copy link
Member

EvanPurkhiser commented Aug 12, 2020

👀

@EvanPurkhiser EvanPurkhiser force-pushed the evanpurkhiser/build-add-36-to-build-matrix branch 10 times, most recently from 6be1973 to 64cfe4c Aug 12, 2020
@EvanPurkhiser EvanPurkhiser mentioned this pull request Aug 12, 2020
@cclauss
Copy link
Contributor

cclauss commented Aug 12, 2020

Travis CI build config validation issues to resolve:
Screenshot 2020-08-12 at 07 52 50

@EvanPurkhiser
Copy link
Member Author

EvanPurkhiser commented Aug 12, 2020

ooh, thanks @cclauss :)

@EvanPurkhiser EvanPurkhiser force-pushed the evanpurkhiser/build-add-36-to-build-matrix branch 2 times, most recently from 451aab0 to 92c3c87 Aug 12, 2020
@EvanPurkhiser
Copy link
Member Author

EvanPurkhiser commented Aug 12, 2020

I think they should be fixed now!

Copy link
Contributor

cclauss left a comment

LGTM. I would encourage you to add a Python 3.8 test even if you do not yet plan to use it because there were some key changes in Py37 and Py38 and it would be good to catch them early.

Python 3.9 is released in about one month.

@EvanPurkhiser
Copy link
Member Author

EvanPurkhiser commented Aug 12, 2020

Yeah sounds good. We're targeting 3.6 for sentry right now and making good progress, then we'll be able to start making those incremental jumps. It's definitely been a struggle to get there, but we're getting close!

@EvanPurkhiser EvanPurkhiser requested a review from getsentry/python-apocalypse Aug 12, 2020
@BYK
BYK approved these changes Aug 12, 2020
rb/ketama.py Outdated Show resolved Hide resolved
@EvanPurkhiser EvanPurkhiser force-pushed the evanpurkhiser/build-add-36-to-build-matrix branch from 92c3c87 to 977c4b2 Aug 12, 2020
rb/ketama.py Show resolved Hide resolved
@EvanPurkhiser EvanPurkhiser merged commit 5a84b7f into master Aug 12, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@EvanPurkhiser EvanPurkhiser deleted the evanpurkhiser/build-add-36-to-build-matrix branch Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.