Skip to content

Removing python 3.5 testing as well as changing required python to 3.6+#8955

Merged
saimn merged 2 commits into
astropy:masterfrom
bsipocz:remove_py35
Jul 4, 2019
Merged

Removing python 3.5 testing as well as changing required python to 3.6+#8955
saimn merged 2 commits into
astropy:masterfrom
bsipocz:remove_py35

Conversation

@bsipocz

@bsipocz bsipocz commented Jul 3, 2019

Copy link
Copy Markdown
Member

Since v4.0 LTS will require python 3.6+ we can remove testing older versions now.

Merging this now means that PRs assuming python 3.6+ compatibility can move forward.

@bsipocz bsipocz added the Release label Jul 3, 2019
@bsipocz bsipocz added this to the v4.0 milestone Jul 3, 2019
@bsipocz bsipocz added the zzz 💤 merge-when-ci-passes Do not use: We have auto-merge option now. label Jul 3, 2019
@taldcroft

Copy link
Copy Markdown
Member

F-strings woo woo!!!

@bsipocz

bsipocz commented Jul 4, 2019

Copy link
Copy Markdown
Member Author

I've also added a quick commit here that specifies the doc file formats to avoid the UserWarning coming from pytest-doctestplus. No need to test it as we don't display the warnings anyway.

@bsipocz

bsipocz commented Jul 4, 2019

Copy link
Copy Markdown
Member Author

F-strings woo woo!!!

🎉 although I think it's better to do those in a follow-up PR. The purpose here is really just to make it possible to get on with PRs that are waiting for this removal.

@saimn saimn merged commit f436216 into astropy:master Jul 4, 2019
@saimn

saimn commented Jul 4, 2019

Copy link
Copy Markdown
Contributor

Thanks @bsipocz! Let's use f-strings now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release zzz 💤 merge-when-ci-passes Do not use: We have auto-merge option now.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants