-
Notifications
You must be signed in to change notification settings - Fork 73
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
chore: Bump Nox #2865
base: main
Are you sure you want to change the base?
chore: Bump Nox #2865
Conversation
Reviewer's Guide by SourceryThis PR updates the nox dependency to the latest version, which includes changes to how nox is invoked and how dependencies are loaded. It also updates the cookiecutter e2e workflow to include the requirements.txt file. Flow diagram of updated Nox dependency changesflowchart LR
subgraph Before
A1[nox.project.load_toml
with explicit path] --> B1[Load dependency groups]
C1[uv tool run] --> D1[Execute tools
cookiecutter/twine/pre-commit]
end
subgraph After
A2[nox.project.load_toml
without path] --> B2[Load dependency groups]
C2[uvx] --> D2[Execute tools
cookiecutter/twine/pre-commit]
end
style Before fill:#f9f,stroke:#333
style After fill:#9f9,stroke:#333
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
CodSpeed Performance ReportMerging #2865 will not alter performanceComparing Summary
|
232bf4f
to
40c91a5
Compare
40c91a5
to
9ad5fb2
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2865 +/- ##
==========================================
- Coverage 91.37% 91.33% -0.04%
==========================================
Files 62 62
Lines 5203 5203
Branches 675 675
==========================================
- Hits 4754 4752 -2
- Misses 317 319 +2
Partials 132 132 ☔ View full report in Codecov by Sentry. |
9ad5fb2
to
e02f669
Compare
@sourcery-ai review |
* chore: Require uv be used with Nox * Use uv more broadly * Move `UV_CONSTRAINT` to the top * Remove redundant installation of griffe
e02f669
to
4d690e2
Compare
Summary by Sourcery
Upgrade Nox to the latest version, addressing issues with uvx and replay files, and improving dependency management.
Build:
uvx
instead ofuv tool run
, simplifying command execution.CI:
📚 Documentation preview 📚: https://meltano-sdk--2865.org.readthedocs.build/en/2865/