-
Notifications
You must be signed in to change notification settings - Fork 6
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
ci: Update Python versions we test against. #215
base: main
Are you sure you want to change the base?
Conversation
Closed via #216 |
Not solved by #216 . Dependency conflicts caused by |
|
Please format your code with black: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #215 +/- ##
==========================================
- Coverage 80.51% 80.49% -0.02%
==========================================
Files 102 102
Lines 7966 7964 -2
==========================================
- Hits 6414 6411 -3
- Misses 1552 1553 +1 ☔ View full report in Codecov by Sentry. |
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,
not sure if we want to roll back the pysam thing in the future, maybe make an issue?
.github/workflows/main.yml
Outdated
# We are missing bioconda pysam packages for 3.11 and 3.12, cf. | ||
# https://github.com/bioconda/bioconda-recipes/issues/37805 | ||
# - name: Save time by installing packages via mamba. | ||
# run: mamba install -y pysam |
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.
Maybe we should open an issue, so this doesn't get lost?
Currently blocked by:
|
We need a new biomedsheets PyPI release. |
No description provided.