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

CI: inih now available for Cygwin #2468

Merged
merged 1 commit into from
Jan 18, 2023
Merged

CI: inih now available for Cygwin #2468

merged 1 commit into from
Jan 18, 2023

Conversation

kmilos
Copy link
Collaborator

@kmilos kmilos commented Jan 18, 2023

Thank you @me-and

Also w/ some cleanup to make MSYS2 and Cygwin jobs more similar.

@codecov
Copy link

codecov bot commented Jan 18, 2023

Codecov Report

Merging #2468 (fe327e1) into main (d5afb08) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2468   +/-   ##
=======================================
  Coverage   62.08%   62.08%           
=======================================
  Files         121      121           
  Lines       22830    22830           
  Branches    11212    11212           
=======================================
  Hits        14175    14175           
  Misses       6448     6448           
  Partials     2207     2207           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

kevinbackhouse
kevinbackhouse previously approved these changes Jan 18, 2023
@mergify mergify bot dismissed kevinbackhouse’s stale review January 18, 2023 12:17

Pull request has been modified.

@kmilos
Copy link
Collaborator Author

kmilos commented Jan 18, 2023

Not sure about the failing test now, can't test locally unfortunately... I guess it might have to do w/ actually making it now use the MSYS2 and Cygwin Python in their respective environments instead of the Windows runner installed native CPython...

Edit: Cygwin had -DBUILD_SAMPLES=OFF by default before. I could turn it off again, unless someone can shed some light on the iotest failure?

@kmilos kmilos merged commit ed67372 into main Jan 18, 2023
@mergify mergify bot deleted the ci_windows_inih branch January 18, 2023 18:00
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.

2 participants