pagure

#5031 CI: install new deps after merging the PR under test
Closed by wombelix. Opened by abompard.
abompard/pagure ci-deps  into  master

Download 5031.patch

This PR reorganises the way tests are run in CI to checkout the correct git branch as early as possible, before dependencies are parsed and installed.
This way if a PR introduces new dependencies, they will be installed in CI and not cause a failure.

I've commented this out because I don't think building the binary is necessary to run the tests. If I'm wrong on that I'll uncomment it.

rebased onto 8968cf679f391b4ce781b917c3ae038d4fcb3958

pretty please pagure-ci rebuild

Metadata Update from @wombelix:
- Request assigned

Thanks @abompard for your contribution. After reviewing your changes and matching them with the logic we have today for the unit test containers, I think you PR was mainly implemented in the meantime and the last missing piece add by https://pagure.io/pagure/pull-request/5464 and https://pagure.io/pagure/pull-request/5465. Therefore I close this PR.

Pull-Request has been closed by wombelix
Metadata
wombelix
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.