Skip to content

Fix package installation failures for static workflow#146

Merged
bschilder merged 2 commits intomasterfrom
static-fix
Dec 17, 2025
Merged

Fix package installation failures for static workflow#146
bschilder merged 2 commits intomasterfrom
static-fix

Conversation

@HDash
Copy link
Member

@HDash HDash commented Dec 5, 2025

No description provided.

@HDash HDash self-assigned this Dec 5, 2025
@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.75%. Comparing base (218fa5f) to head (c706458).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   85.75%   85.75%           
=======================================
  Files          59       59           
  Lines        1200     1200           
=======================================
  Hits         1029     1029           
  Misses        171      171           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bschilder
Copy link
Collaborator

Thanks @HDash! All good to be merged ?

@HDash
Copy link
Member Author

HDash commented Dec 12, 2025

Hey @bschilder, reverting the Bioc version for static tests to RELEASE seems to fix the issues. Not sure why R dependency package downloads fail with the devel version.

I suspect it’s a devel issue, but switching to RELEASE just to fix this might be a temporary bandaid, unless the stars align and the devel sorts itself out.

@bschilder
Copy link
Collaborator

Hey @bschilder, reverting the Bioc version for static tests to RELEASE seems to fix the issues. Not sure why R dependency package downloads fail with the devel version.

I suspect it’s a devel issue, but switching to RELEASE just to fix this might be a temporary bandaid, unless the stars align and the devel sorts itself out.

devel is more likely to have some broken deps bc developers are usually experimenting w new code there. Still, it can be helpful to "look ahead" at potential issues. I think for the purposes of the static workflow using RELEASE should be fine for now.

@bschilder bschilder merged commit 16cc86c into master Dec 17, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants