Skip to content

TST: Using GitHub Actions for CI testing #1691

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.

Sign up for GitHub

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

Jump to bottom
Merged
QuLogic merged 3 commits into from
Dec 22, 2020
Merged

TST: Using GitHub Actions for CI testing #1691

QuLogic merged 3 commits into from
Dec 22, 2020

Conversation

@greglucas
Copy link
Contributor

@greglucas greglucas commented Dec 11, 2020

Adding the yaml configuration file to use GitHub Actions for testing.

Rationale

TravisCI may no longer be supporting open-source the way it previously did with free resources.

Implications

New test integration platform and needing to learn the GitHub Actions yaml language and platform.

QuLogic
QuLogic reviewed Dec 16, 2020
View reviewed changes
@greglucas greglucas force-pushed the github_actions branch 2 times, most recently from fa4ad57 to f88cc41 Compare December 16, 2020 15:13
@greglucas
Copy link
Contributor Author

greglucas commented Dec 16, 2020

Coveralls on GitHub reports 85%, but on TravisCI it reports 90% which is the reason for the drop once I removed the TravisCI builds.

@dopplershift
Copy link
Contributor

dopplershift commented Dec 16, 2020

Was that a bug or did we suddenly stop running some important config? Or do we have code we can remove because we're not supporting some combination we were hacking around?

@greglucas
Copy link
Contributor Author

greglucas commented Dec 16, 2020

It looks like we are now testing a new file: _geodesic.pyx in https://coveralls.io/builds/35765159/source?filename=lib/cartopy/geodesic/_geodesic.pyx vs https://coveralls.io/builds/35766611 that only tests the __init__.py file from the geodesic package. That new file isn't fully covered which is what is causing the overall coverage to decrease. So, nothing has actually decreased as far as I can tell, just now testing more things.

@greglucas
TST: Using GitHub Actions for CI testing
b23ccec 
Adding the yaml configuration file to use GitHub Actions for testing.
@greglucas
TST: Removing TravisCI from the testing framework
ef77594 
Removing TravisCI due to the flaky builds sometimes
reporting failures.
@greglucas greglucas force-pushed the github_actions branch 3 times, most recently from aef68e2 to f78d6ef Compare December 21, 2020 18:35
@greglucas
MAINT: Adding automated release uploads to GitHub Actions
bbcf26b 
Adding source distribution build and PyPI release actions.
QuLogic
QuLogic approved these changes Dec 22, 2020
View reviewed changes
Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

Best to just start testing with it.

@QuLogic QuLogic merged commit 7237260 into SciTools:master Dec 22, 2020
@greglucas greglucas deleted the github_actions branch December 22, 2020 14:51
@rcomer rcomer mentioned this pull request Jun 11, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dopplershift dopplershift dopplershift approved these changes

@QuLogic QuLogic QuLogic approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@greglucas @dopplershift @QuLogic