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

[TST] GitHub Actions only; Migrate away from Travis/Appveyor #453

Merged
merged 7 commits into from
Mar 26, 2020

Conversation

jGaboardi
Copy link
Member

@jGaboardi jGaboardi commented Mar 25, 2020

This PR embodies a large break in the current standard PySAL testing schema.

Migration Highlights

  • significantly less complex
  • tests ubuntu, mac, windows
  • tests Python 3.6, 3.7, 3.8
  • tests both hard and optional requirements
  • tests with both the current libpysal release and the current master
  • performs code-cov
  • run nightly tests
  • negates dual dependency on Travis and Appveyor for testing

@jGaboardi jGaboardi added this to the Release v1.5.0 milestone Mar 25, 2020
@jGaboardi jGaboardi self-assigned this Mar 25, 2020
@codecov
Copy link

codecov bot commented Mar 25, 2020

Codecov Report

Merging #453 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #453   +/-   ##
=======================================
  Coverage   97.85%   97.85%           
=======================================
  Files           3        3           
  Lines        1120     1120           
  Branches      269      269           
=======================================
  Hits         1096     1096           
  Misses          6        6           
  Partials       18       18

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52e4525...886a2da. Read the comment docs.

@jGaboardi
Copy link
Member Author

@sjsrey I'm not sure why random builds in the matrix hang/stall, but I have a question about in to the GitHub Community Forum. The stalling builds are generally different each time, with some workflow runs passing completely without stalled builds. This PR successfully disabled Travis, but Appveyor is still being triggered (even though I disabled the webhook). I think we should merge this see how it goes.

@jGaboardi jGaboardi requested a review from sjsrey March 25, 2020 20:35
@jGaboardi jGaboardi mentioned this pull request Mar 25, 2020
47 tasks
@jGaboardi
Copy link
Member Author

All GitHub actions pass after rerunning several times. Merging. Will try to debug stalls/hangups later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant