Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Apr 22, 2016

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
Copy link
Contributor

rimey commented Apr 22, 2016

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

@tseaver
Copy link
Contributor

tseaver commented Apr 22, 2016

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

@dhermes
Copy link
Contributor Author

dhermes commented Apr 22, 2016

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

@tseaver
Copy link
Contributor

tseaver commented Apr 22, 2016

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

@dhermes
Copy link
Contributor Author

dhermes commented Apr 22, 2016

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
Copy link
Contributor

tseaver commented Apr 22, 2016

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

@dhermes
Copy link
Contributor Author

dhermes commented Apr 22, 2016

Nope. Just tox.ini for this PR.

@tseaver
Copy link
Contributor

tseaver commented Apr 22, 2016

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
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
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