Migrate to uv #1098
Migrate to uv #1098
Conversation
michelletran-codecov
changed the title
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #1098 +/- ##
=======================================
Coverage 97.25% 97.25%
=======================================
Files 454 454
Lines 37335 37337 +2
=======================================
+ Hits 36310 36312 +2
Misses 1025 1025
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. 📢 Thoughts on this report? Let us know! |
michelletran-codecov
force-pushed
the
migrate_uv
branch
3 times, most recently
from
a9bccb6 to
0d0d3c1
Compare
This migrates the dependencies from requirements.in/txt to pyproject.toml and uv.lock.
michelletran-codecov
force-pushed
the
migrate_uv
branch
from
2f7f2f0 to
e439555
Compare
|
This PR depends on codecov/gha-workflows#44 |
michelletran-codecov
had a problem deploying
to
staging
GitHub Actions
Failure
.github/workflows/ci.yml
Outdated
| @@ -44,7 +46,7 @@ jobs: | |||
| test: | |||
| name: Test | |||
| needs: [build] | |||
| uses: codecov/gha-workflows/.github/workflows/run-tests.yml@v1.2.30 | |||
| uses: codecov/gha-workflows/.github/workflows/run-tests.yml@1787b7a62b96639a32ebfa966ef67cf852033f16 | |||
There was a problem hiding this comment.
curious what's the reasoning to change from versions to sha's?
There was a problem hiding this comment.
Oh, this will be a version, but I just need to merge and release codecov/gha-workflows#44 first!
The push-env workflow requires that the images be built in a specific way to be able to reference the cached images.
michelletran-codecov
force-pushed
the
migrate_uv
branch
from
4cfe18e to
547a177
Compare
Just updating the rest of the references to ensure that they're the most recent version.
README.md
Outdated
| @@ -85,13 +85,14 @@ If you are unsure whether you need to change that version at a given moment, the | |||
|
|
|||
| ## Upgrading Dependencies | |||
|
|
|||
| This repository uses `pip-tools` to manage dependencies, so make sure you've installed it with `pip install pip-tools`. To add or update dependencies, change `requirements.in`, Then run | |||
| This repository uses `uv` to manage dependencies. | |||
| Make your dependency updates to `pyproject.toml` then run: | |||
|
|
|||
| ``` | |||
| make update-requirements | |||
There was a problem hiding this comment.
I don’t think this is necessary anymore. The make command still uses uv pip compile. uv sync will automatically sync the installed dependencies and lockfile depending on the pyproject.toml definition.
We can just run `uv sync` directly.
michelletran-codecov
force-pushed
the
migrate_uv
branch
from
7847dd1 to
fe03d39
Compare
This migrates the dependencies from requirements.in/txt to pyproject.toml and uv.lock.
This also changes the way that the Docker containers are built and how dependency updates are to be done going forward. Please review the updated instructions on how to update or add packages with
uv.Legal Boilerplate
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.