switch to r-lib/actions/setup-r-dependencies@v2

[m7pr]

Part of https://github.com/insightsengineering/coredev-tasks/issues/609

[cicdguy]

Looks good

[github-actions]

Unit Tests Summary

3 tests   2 ✅  4s ⏱️
3 suites  1 💤
1 files    0 ❌

Results for commit 47dc57f.

♻️ This comment has been updated with latest results.

[pawelru]

These are good changes but not complete as of now.
Also - this can be merged only if Remotes is added in all (!) the repos. Please note that!

[m7pr]

I'll be testing with this teal PR insightsengineering/teal#1466
but will first check your comments and advices

[m7pr]

@pawelru

if Remotes is added in all (!) the repos.

by all, how many we are talking about? all in https://github.com/insightsengineering org?

[pawelru]

@pawelru

if Remotes is added in all (!) the repos.

by all, how many we are talking about? all in https://github.com/insightsengineering org?

All where necessary. Obviously these live in our org and include also tern and family etc.

[m7pr]

@pawelru so far I adjusted possible inputs that are passed to r-lib/actions/setup-r-dependencies@v2 and removed those inputs from example teal PR insightsengineering/teal#1466

[pawelru]

This is a good one and I'm ok with that but please note that it's a breaking change for all the packages that are still using lookup-refs parameter! Hence not approving formally but just saying it's good but others are not ready.

[m7pr]

A test PR in teal insightsengineering/teal#1484 that runs checks via this PR

[pawelru]

OK. This is good to go. Please make sure that all (!) the upstream packages follows - in particular: they might be passing parameters that you are about to remove! This most likely would fail the jobs.

[m7pr]

For now it runs smoothly in the PR created in teal to test that insightsengineering/teal#1484

I also created a card to track down all other packages insightsengineering/nestdevs-tasks#103

[m7pr]

Adding a comment, so it can not be merged.
It should not be merged before other packages are not suited for changes introduced in this PR.
Other packages are tracked in this card insightsengineering/nestdevs-tasks#103