Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

switch CI off travis-ci #439

Closed
willkg opened this issue Feb 21, 2019 · 4 comments · Fixed by #514
Closed

switch CI off travis-ci #439

willkg opened this issue Feb 21, 2019 · 4 comments · Fixed by #514

Comments

@willkg
Copy link
Member

willkg commented Feb 21, 2019

Bleach currently uses Travis CI for thing, but it seems like Travis CI is going to go away. We should switch to something else.

A lot of Mozilla projects use Circle CI, so I think that's probably the best option for now.

@g-k g-k changed the title switch to circleci switch CI off travis-ci Dec 5, 2019
@g-k
Copy link
Collaborator

g-k commented Dec 5, 2019

see also: #331

@willkg
Copy link
Member Author

willkg commented Dec 5, 2019

I was just reading through this post which talks about setting up github actions and testing across linux, macos, and windows:

https://dan.yeaw.me/posts/github-actions-automate-your-python-development-workflow/

Figured maybe that would be helpful.

@g-k
Copy link
Collaborator

g-k commented Dec 5, 2019

Awesome, that looks great! Thanks willkg

Also, I'm +1 on publishing to pypi from CI (so we don't have to manage pypi creds on our laptops).

@hugovk
Copy link
Contributor

hugovk commented Jan 28, 2020

Please see PR #514 for testing and linting on GHA.

You'd need to keep Travis at least for Python 3.7 and 3.8 until #503 is fixed, but some jobs could be removed from Travis.

Some things that Travis has but GHA doesn't, which could be useful: 3.8-dev and 3.9-dev. Also up to 4 different CPU architectures (eg. one is big-endian), but that's probably more useful for projects with C code.

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 a pull request may close this issue.

3 participants