Skip to content

Workaround for https://github.com/r-lib/pak/issues/755 - #1077

Merged
jgabry merged 3 commits into
stan-dev:masterfrom
katrinabrock:workaround-pak-755
Aug 18, 2025
Merged

Workaround for https://github.com/r-lib/pak/issues/755#1077
jgabry merged 3 commits into
stan-dev:masterfrom
katrinabrock:workaround-pak-755

Conversation

@katrinabrock

@katrinabrockkatrinabrock commented Apr 14, 2025

Copy link
Copy Markdown
Contributor

Submission Checklist

  • Run unit tests
  • Declare copyright holder and agree to license (see below)

Summary

This issue: r-lib/pak#755 (also reported here: r-lib/actions#994)
impacts github actions setup-r-dependenceis of ubuntu latest intermittently. I haven't seen it on main yet, but has it me when testing on my fork: https://github.com/katrinabrock/cmdstanr/actions/runs/14448908294/job/40516367645 Thought it would be good to per-emptively prevent this from causing CI failures.

Copyright and Licensing

Please list the copyright holder for the work you are submitting
(this will be you or your assignee, such as a university or company):
Max Planck Institute of Animal Behavior

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the following licenses:

@katrinabrockkatrinabrock mentioned this pull request Apr 14, 2025
2 tasks

@andrjohnsandrjohns left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@katrinabrock

Copy link
Copy Markdown
ContributorAuthor

Looks like this is actually not working. :-/ Following up on the pak thread.

@katrinabrock

Copy link
Copy Markdown
ContributorAuthor

This fix: r-lib/pkgdepends#442 should make the workaround run without error....let's see 🤞.

@codecov-commenter

codecov-commenter commented Aug 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.22%. Comparing base (5f2a8c8) to head (14d81ac).

Additional details and impacted files
@@ Coverage Diff @@## master #1077 +/- ##
=======================================
Coverage 87.22% 87.22% =======================================
Files 14 14 Lines 5973 5973 =======================================
Hits 5210 5210 Misses 763 763 

☔ 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.

@katrinabrock

Copy link
Copy Markdown
ContributorAuthor

@andrjohns@jgabry this can be merged

@jgabry

Copy link
Copy Markdown
Member

Great, thanks @katrinabrock!

@jgabry
jgabry merged commit 175bfb4 into stan-dev:masterAug 18, 2025
11 checks passed
Sign up for freeto 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.

4 participants

@katrinabrock@codecov-commenter@jgabry@andrjohns