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

Adjust CI for Numpy 2.0, Graphviz #854

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Adjust CI for Numpy 2.0, Graphviz #854

merged 2 commits into from
Jun 17, 2024

Conversation

khaeru
Copy link
Member

@khaeru khaeru commented Jun 17, 2024

This PR updates the "pytest" CI workflow to address two causes of failures:

Both of these are temporary work-arounds, are marked accordingly, and should be removed when the respective upstream packages are fixed.

How to review

Read the diff and note that the CI checks all pass.

PR checklist

  • Continuous integration checks all ✅
  • Add or expand tests; coverage checks both ✅ CI changes only.
  • Add, expand, or update documentation.
  • Update release notes.

@khaeru khaeru added the ci Continuous integration label Jun 17, 2024
@khaeru khaeru self-assigned this Jun 17, 2024
Copy link

codecov bot commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.6%. Comparing base (1125579) to head (c9df55e).
Report is 16 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #854   +/-   ##
=====================================
  Coverage   95.6%   95.6%           
=====================================
  Files         46      46           
  Lines       4333    4333           
=====================================
  Hits        4143    4143           
  Misses       190     190           

@khaeru khaeru merged commit fa21aa2 into main Jun 17, 2024
26 checks passed
@khaeru khaeru deleted the ci/numpy-2 branch June 17, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant