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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ci/rubyonrails bugs, using binstubs #2427

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

jamiemccarthy
Copy link

This PR is identical to #2302 with one additional commit, 0321f5f, that switches the CI action to run using binstubs in bin/ rather than bundle exec prefixes.

I am proposing that either this PR or #2302 should be merged, depending on personal preference for how those ruby commands get run. The other PR should be closed. I don't care which 馃槃

This has been verified to work in a personal repo's GitHub Action, namely here in this run:

Run bundle binstubs bundler-audit brakeman rubocop

Run bin/bundler-audit --update
...
No vulnerabilities found

@jamiemccarthy jamiemccarthy requested review from a team as code owners June 8, 2024 16:25
@jamiemccarthy jamiemccarthy changed the title Fix ci/rubyonrails bugs Fix ci/rubyonrails bugs, using binstubs Jun 8, 2024
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.

None yet

1 participant