Skip to content

Conversation

lvh
Copy link
Member

@lvh lvh commented Jun 23, 2014

This makes Travis run a bunch of tox environments. This is good, because it enables moving more behavior into tox. That's good, because it enables running something a lot closer to what CI runs on your own machine.

Refs #127.

.travis.yml Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't this install the package outside the tox env?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh, you're right. Suggestions? Create git master tox environments?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose it would be easier to teach tox about CRYPTOGRAPHY_GIT_MASTER instead.

@lvh
Copy link
Member Author

lvh commented Jun 24, 2014

So, apparently there's no easy way to express matrix-style tox envs yet, so we can't do something much nicer than the old travis config.

I wrote a script to do the work of creating all the testenv specs by taking the cartesian product of all the possibilities:

https://gist.github.com/lvh/5a2e04b9315ffeb20e62

@lvh
Copy link
Member Author

lvh commented Jun 24, 2014

Travis appears to be taking a very long time to get around to testing this :(

lvh added 2 commits June 24, 2014 12:14
coverage does the right thing when you point it at a python file
@lvh
Copy link
Member Author

lvh commented Jun 24, 2014

Cool! Now the tests are actually running properly, and we're down to a single failure, which is an unrelated test failure on 0.9.8k. I had accidentally forgot to add the 0.9.8 build back to "allowed-failures". Fixing that now :)

@lvh
Copy link
Member Author

lvh commented Jun 24, 2014

Apparently I hadn't forgotten, but Travis didn't pick it up. I tried being a bit more explicit this time.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8ffcd62 on lvh:tox-powered-travis into 1110bc9 on pyca:master.

@lvh
Copy link
Member Author

lvh commented Jun 24, 2014

Yay! Ready for review now I guess :)

@reaperhulk
Copy link
Member

obsoleted by #230

@reaperhulk reaperhulk closed this Apr 15, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants