Skip to content

use new images - #265

Open
j-i-l wants to merge 4 commits into
mainfrom
new-container-sources
Open

j-i-l wants to merge 4 commits into
mainfrom
new-container-sources

Conversation

@j-i-l

@j-i-l j-i-l commented Aug 25, 2026

Copy link
Copy Markdown
Member

Recently the r-container updated the source of their base images and changed the selection of container images they offer: furrer-lab/r-containers#69

We need to make sure to use containers for our checks that are regularly updated and maintained upstream, i.e. r-container and then r-hub.

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.58%. Comparing base (d7b018c) to head (62a7fd9).
⚠️ Report is 65 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #265      +/-   ##
==========================================
+ Coverage   66.30%   67.58%   +1.27%     
==========================================
  Files          53       50       -3     
  Lines       10399    10799     +400     
==========================================
+ Hits         6895     7298     +403     
+ Misses       3504     3501       -3     

see 3 files with indirect coverage changes

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

@j-i-l

j-i-l commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

valgrind check run as expected, but we need to tune somewhat the conditions for exit=1, currently most fail with 64 bytes definitely lost: glibc/R dynamic-loader allocation which is definitely not something related to abn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant