Skip to content

add tox config to test Python 3.5 to 3.8#45

Open
BurnzZ wants to merge 9 commits intomasterfrom
add-tox
Open

add tox config to test Python 3.5 to 3.8#45
BurnzZ wants to merge 9 commits intomasterfrom
add-tox

Conversation

@BurnzZ
Copy link
Copy Markdown
Contributor

@BurnzZ BurnzZ commented Aug 6, 2020

@akshayphilar I'm adding some tox configs since we need to ensure that the new functionalities we will be implementing in https://github.com/scrapinghub/shublang/issues?q=is%3Aissue+is%3Aopen+label%3A%22good+first+issue%22 correctly runs across different Python 3 versions.

@jsargiot, I'm quite unfamiliar with CircleCI as SH uses Travis CI in majority of its repos. Could you review if I'm invoking tox correctly?

For Travis CI, we usually configure it via https://github.com/scrapy/itemloaders/blob/master/.travis.yml#L8-L23 which results in https://travis-ci.com/github/scrapy/itemloaders/builds/178576103, testing across the declared Python environments.

EDIT: Looks like the new tox config correctly runs in Circle CI as can be seen in this build. However, it would seem that some of the Python envs are not setup yet in it.

image

Thanks!

@BurnzZ BurnzZ requested review from akshayphilar and jsargiot August 6, 2020 05:01
@BurnzZ BurnzZ self-assigned this Aug 6, 2020
Copy link
Copy Markdown
Member

@akshayphilar akshayphilar left a comment

Choose a reason for hiding this comment

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

LGTM @BurnzZ

@jsargiot
Copy link
Copy Markdown
Contributor

jsargiot commented Aug 7, 2020

It seems this is the best approach to have different python versions: https://ben.fogbutter.com/2016/02/20/testing-multiple-python-versions-on-circleci.html

@BurnzZ BurnzZ force-pushed the add-tox branch 6 times, most recently from c31b319 to 3b82544 Compare August 7, 2020 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants