Skip to content

Conversation

@msberends
Copy link

@msberends msberends commented Jun 6, 2025

I had thought this ought to work:

- uses: r-lib/actions/setup-r@v2
  with:
    r-version: ${{ matrix.config.r }}
    use-public-rspm: true
    extra-repositories:
      - 'https://myrepo1.r-universe.dev'
      - 'https://myrepo2.r-universe.dev'

But it doesn't - it should apparently be:

- uses: r-lib/actions/setup-r@v2
  with:
    r-version: ${{ matrix.config.r }}
    use-public-rspm: true
    extra-repositories: 'https://myrepo1.r-universe.dev, https://myrepo2.r-universe.dev'

This was not clear from the documentation, so I updated that.

Still think that the element should be pasted together internally to allow common YAML language like my first example.

@gaborcsardi
Copy link
Member

Still think that the element should be pasted together internally to allow common YAML language like my first example.

Sure, a PR is welcome for that. 😄

default: false
extra-repositories:
description: 'One or more extra CRAN-like repositories to include in the `repos` global option'
description: 'One or more extra CRAN-like repositories to include in the `repos` global option. This must be a single character; multiple repos can be set using a comma: "https://repo1.org, https://repo2.org".'
Copy link
Member

Choose a reason for hiding this comment

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

Thanks! Can you please change this to avoid the very long line?

Copy link
Author

Choose a reason for hiding this comment

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

Absolutely - I'll update it

@msberends
Copy link
Author

Still think that the element should be pasted together internally to allow common YAML language like my first example.

Sure, a PR is welcome for that. 😄

Good point, I'll try to familiarise myself with .ts files!

@msberends
Copy link
Author

I believe this should do it, but please check critically, I've never worked in TypeScript before.

Should I also update the README.md or will that happen automatically?

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