Skip to content

Use --force-reinstall in integration test and skip wheel#4565

Merged
abravalheri merged 2 commits intomainfrom
fix-integration-test
Aug 13, 2024
Merged

Use --force-reinstall in integration test and skip wheel#4565
abravalheri merged 2 commits intomainfrom
fix-integration-test

Conversation

@abravalheri
Copy link
Copy Markdown
Contributor

Summary of changes

In integration test:

  • Replace pip install -I with pip install --force-reinstall
  • Do not install wheel as it is no longer a build dependency (it also no longer gets automatically installed in the build env in regular workflows).

Closes #4564

Pull Request Checklist

@abravalheri abravalheri changed the title Use --force-reinstall in integration test and skip wheel Use --force-reinstall in integration test and skip wheel Aug 13, 2024
@abravalheri
Copy link
Copy Markdown
Contributor Author

Integration tests should run in https://github.com/pypa/setuptools/actions/runs/10369805360, so we can see if this fixes the problem or not.

@abravalheri
Copy link
Copy Markdown
Contributor Author

Integration tests pass after the changes: https://github.com/pypa/setuptools/actions/runs/10369805360/job/28707136245.

@abravalheri abravalheri marked this pull request as ready for review August 13, 2024 14:24
@abravalheri abravalheri merged commit 6fcce38 into main Aug 13, 2024
@abravalheri abravalheri deleted the fix-integration-test branch August 13, 2024 14:25
@jaraco
Copy link
Copy Markdown
Member

jaraco commented Aug 13, 2024

I re-ran the failed jobs for v72.2.0, since the release didn't go out as planned, and it succeeded.

@jaraco
Copy link
Copy Markdown
Member

jaraco commented Aug 13, 2024

Because the issue is intermittent, we'll want to keep an eye out to see if it happens again. Presumably this change will fix the issue. Thanks for working on it!

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.

Integration tests for brotli are failing

2 participants