-
Notifications
You must be signed in to change notification settings - Fork 4
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
Developer tools #134
Developer tools #134
Conversation
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.
Could you also update the other GitHub Actions workflows please?
Done. Please let me know whether we should keep line 47 in |
That was to be able to use development versions of rstan. Since we are sticking only to CRAN versions, I guess we don't need it anymore |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #134 +/- ##
==========================================
- Coverage 76.25% 76.16% -0.09%
==========================================
Files 10 10
Lines 1617 1611 -6
==========================================
- Hits 1233 1227 -6
Misses 384 384 ☔ View full report in Codecov by Sentry. |
- tools/check.env - Add exceptions for inconsequential notes
I added `extra-repositories: https://mc-stan.org/r-packages/` in line 47 to keep the previous configuration.
Co-authored-by: Hugo Gruson <Bisaloo@users.noreply.github.com>
Clean diff is here: https://github.com/epiverse-trace/serofoi/compare/developer-tools2?expand=1. Can I force push it here? Alternatively, are you able to rebase yourself? You'll have to run |
Sure, please do. Sorry for the trouble. |
94f0e5c
to
e7e32cd
Compare
This PR updates the package template files:
In accordance to the discussion in November 2023 spring cleaning #101