chore: Update dependencies#103
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughPinned DuckDB core and its extensions to the 1.4.x LTS line and added ujson and idna to [project].dependencies in pyproject.toml. Extended the CI pip-audit Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
📦 Python package built successfully!
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #103 +/- ##
=======================================
Coverage 74.09% 74.09%
=======================================
Files 95 95
Lines 5678 5678
Branches 843 843
=======================================
Hits 4207 4207
Misses 1195 1195
Partials 276 276
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
🚀 Review App Deployment Started
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@pyproject.toml`:
- Around line 58-60: Consolidate the redundant duckdb dependency entries in
pyproject.toml: replace the three duplicate lines specifying
"duckdb>=1.4.2,<1.5.0" with a single dependency line (e.g.,
"duckdb>=1.4.2,<1.5.0") and remove the overlapping environment markers
("python_version >= '3.12'" and "python_version >= '3.13'") so only one
canonical entry for duckdb remains.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: a85132d5-3664-4b92-9b99-efd8f313ef31
⛔ Files ignored due to path filters (1)
poetry.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
.github/workflows/ci.ymlpyproject.toml
Summary by CodeRabbit