-
Notifications
You must be signed in to change notification settings - Fork 0
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: unpin pybedlite dependency #98
Conversation
WalkthroughThe changes in the pull request primarily involve updates to the No new dependencies have been added or removed, and the existing development dependencies remain unchanged. The overall structure and formatting of the The build system requirements specify that 📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
pyproject.toml (1)
Line range hint
38-45
: Consider consistent dependency pinning strategy.Other dependencies mix pinned (fgpyo=0.7.1) and unpinned (^0.22.1, ^4.1.0) versions. Establish consistent policy.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
pyproject.toml
(1 hunks)
🔇 Additional comments (2)
pyproject.toml (2)
Line range hint 3-3
: Version bump looks appropriate.
2.2.0 version increment aligns with dependency changes.
38-38
: Verify pybedlite compatibility across minor versions.
Unpinning to ^1.0.0 allows minor updates. Need to ensure backward compatibility.
✅ Verification successful
Unpinning pybedlite to ^1.0.0 is safe.
Limited usage shows only stable Interval and OverlapDetector imports. Version history shows steady progression (0.0.1 -> 1.0.0) with recent 1.0.0 release, indicating API stability.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Check available pybedlite versions and their release dates
curl -s https://pypi.org/pypi/pybedlite/json | jq -r '.releases | keys[] as $k | "\($k)\t\(.[$k][0].upload_time)"' | sort -V
# Check for any breaking changes in pybedlite changelog
gh api repos/fulcrumgenomics/pybedlite/releases | jq -r '.[].body' | grep -i "break"
Length of output: 315
Script:
#!/bin/bash
# Check for usage of pybedlite in the codebase
rg "import.*pybedlite|from.*pybedlite" -l
# Check specific pybedlite function/class usage
rg "pybedlite\." -A 2
# Look for any test files related to pybedlite
fd "test.*\.py" --exec grep -l "pybedlite"
Length of output: 2349
a2d9a7c
to
07e8083
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #98 +/- ##
=======================================
Coverage ? 96.66%
=======================================
Files ? 26
Lines ? 1708
Branches ? 189
=======================================
Hits ? 1651
Misses ? 31
Partials ? 26 ☔ View full report in Codecov by Sentry. |
For post-v1 dependencies that strictly follow SemVer, I go one step further and caret pin to the minor version. So for pybedlite, I'd do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Similar to #97 , this PR unpins the dependency on pybedlite. I am opening two separate PRs in case there is an argument for one dependency to be pinned but not the other.
93658a0
to
27c5d4c
Compare
Similar to #97 , this PR unpins the dependency on pybedlite.
I am opening two separate PRs in case there is an argument for one dependency to be pinned but not the other.