Skip to content

Revert "Use uv sync to set up dependencies in ci.yaml"#875

Merged
nathanjmcdougall merged 1 commit intomainfrom
revert-874-252-revisit-the-hacky-requirementstxt-approach-in-ciyml2
Jul 21, 2025
Merged

Revert "Use uv sync to set up dependencies in ci.yaml"#875
nathanjmcdougall merged 1 commit intomainfrom
revert-874-252-revisit-the-hacky-requirementstxt-approach-in-ciyml2

Conversation

@nathanjmcdougall
Copy link
Copy Markdown
Collaborator

Reverts #874

@nathanjmcdougall nathanjmcdougall merged commit 9a36ebd into main Jul 21, 2025
17 checks passed
@nathanjmcdougall nathanjmcdougall deleted the revert-874-252-revisit-the-hacky-requirementstxt-approach-in-ciyml2 branch July 21, 2025 01:21
nathanjmcdougall added a commit that referenced this pull request Jul 21, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Jul 21, 2025

CodSpeed Instrumentation Performance Report

Merging #875 will not alter performance

Comparing revert-874-252-revisit-the-hacky-requirementstxt-approach-in-ciyml2 (befcf77) with main (9fab283)1

Summary

✅ 2 untouched benchmarks

Footnotes

  1. No successful run was found on main (ba670d5) during the generation of this report, so 9fab283 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

nathanjmcdougall added a commit that referenced this pull request Oct 10, 2025
* Revert "Revert "Use `uv sync` to set up dependencies in `ci.yaml` (#874)" (#875)"

This reverts commit 9a36ebd.

* Fail CI if there are codecov upload errors

* Add `--cov-report=xml` for report generation
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.

1 participant