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

Update helpers v3.2.1 #291

Merged
merged 3 commits into from Jul 30, 2019
Merged

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Jul 2, 2019

This is an exact copy of #287 that has been reset from master given that we need to keep python2 support just for a little longer.

@bsipocz bsipocz added this to the 0.6 milestone Jul 2, 2019
@bsipocz
Copy link
Member Author

bsipocz commented Jul 2, 2019

This will obviously need the rebase before merging, e.g. the pytest5 issues has been addressed in #288

@cdeil
Copy link
Member

cdeil commented Jul 13, 2019

@bsipocz - Please trigger a new CI build here.

@bsipocz
Copy link
Member Author

bsipocz commented Jul 13, 2019

@cdeil - shall I rebase, too?

@cdeil
Copy link
Member

cdeil commented Jul 13, 2019

ok

@cdeil
Copy link
Member

cdeil commented Jul 13, 2019

Ah, I see, this PR drops Python 2 support, that wasn't clear to me before.

So this should not be merged for now.

When to drop Python 2 support is being discussed in #241

@bsipocz
Copy link
Member Author

bsipocz commented Jul 14, 2019

Yes, thus is milestoned to 0.6, that is in my understanding the first version that will be python 3 only.

@keflavich
Copy link
Contributor

I'm 👍 on merging this and ditching py2 support from 0.5 forward

@bsipocz
Copy link
Member Author

bsipocz commented Jul 30, 2019

push the button then ;)

@keflavich keflavich merged commit b9f6ad0 into astropy:master Jul 30, 2019
@keflavich keflavich mentioned this pull request Jul 30, 2019
@cdeil cdeil modified the milestones: 0.6, 0.5 Oct 21, 2019
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.

None yet

3 participants