Skip to content

Removing py26 and pandas from default tox environments.#1739

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:tox-default-reduce
Apr 22, 2016
Merged

Removing py26 and pandas from default tox environments.#1739
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:tox-default-reduce

Conversation

@dhermes

@dhermes dhermes commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

FYI @rimey I'm not sure if you intended to make py27-pandas one of the default environments but the install takes quite a bit of time / it isn't strictly necessary.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 22, 2016
@rimey

rimey commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

No problem. I'm happy running tox -e p27-pandas manually whenever I touch that code.

@tseaver

tseaver commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

@dhermes do you mean to remove py26 from Travis as well?

@dhermes

dhermes commented Apr 22, 2016

Copy link
Copy Markdown
Contributor Author

Yes I did mean to remove it. In light of #1607 (August 1, 2016 is when we drop support).

@tseaver

tseaver commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

@dhermes this PR includes only changes to tox.ini.

@dhermes

dhermes commented Apr 22, 2016

Copy link
Copy Markdown
Contributor Author

I know. You asked if I intended to remove py26 from the tox defaults. Yes I did. The coming deprecation is why I did.

@tseaver

tseaver commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

Do you mean to remove py26 from .travis.yml as well as tox.ini?

@dhermes

dhermes commented Apr 22, 2016

Copy link
Copy Markdown
Contributor Author

Nope. Just tox.ini for this PR.

@tseaver

tseaver commented Apr 22, 2016

Copy link
Copy Markdown
Contributor

OK, LGTM

@dhermes dhermes merged commit dfc8699 into googleapis:master Apr 22, 2016
@dhermes dhermes deleted the tox-default-reduce branch April 22, 2016 19:31
parthea added a commit that referenced this pull request Nov 24, 2025
parthea pushed a commit that referenced this pull request Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants