Skip to content
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

R fix dependency installation #3501

Merged
merged 5 commits into from
Apr 10, 2023
Merged

Conversation

tonyk7440
Copy link
Contributor

@tonyk7440 tonyk7440 commented Oct 28, 2022

Fixes #3473
I tried using the updated action here to test the updates but it doesn't seem to be retrieving them. I believe it should fix the problem but I am unsure how to test that to make sure. Any advice here is appreciated

Proposed Changes

  1. Installs the remotes package
  2. Installs the current package and its dependencies using the remotes package

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

  • Label as breaking if this is a large fundamental change
  • Label as either automation, bug, documentation, enhancement, infrastructure, or performance

@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this pull request should stay open, please remove the O: stale 🤖 label or comment on the pull request.

If you're a maintainer, you can stop the bot to mark this issue as stale in the future by adding the O: backlog 🤖 label`.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Nov 28, 2022
@tonyk7440
Copy link
Contributor Author

Not stale, waiting on review of PR

@github-actions github-actions bot removed the O: stale 🤖 Stale issue/pr label Nov 28, 2022
@ferrarimarco ferrarimarco added the O: backlog 🤖 Backlog, stale ignores this label label Dec 22, 2022
@ferrarimarco ferrarimarco added the bug Something isn't working label Jan 5, 2023
tonyk7440 added a commit to tonyk7440/super-linter that referenced this pull request Jan 16, 2023
@tonyk7440
Copy link
Contributor Author

Hi @ferrarimarco , could you perhaps review again?

Thanks!

@lindluni lindluni merged commit 516b3f8 into super-linter:main Apr 10, 2023
@cicdguy
Copy link
Contributor

cicdguy commented Oct 2, 2023

Unfortunately this is breaking workflows across R repos everywhere else. Will revert this.
You do not need an R package to be installed for the lintr to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working O: backlog 🤖 Backlog, stale ignores this label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

R package linting does not install dependencies raising errors
4 participants