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

Don't generate bin/rubocop if Rails has already provided it #103

Merged
merged 1 commit into from
Sep 7, 2024

Conversation

mattbrictson
Copy link
Owner

Starting with Rails 7.2, rails new generates a custom bin/rubocop binstub that contains some optimizations. If we generate a standard binstub using bundle binstub rubocop, this will inadvertently clobber those optimizations.

Fix by generating the binstub only when it doesn't already exist.

@mattbrictson mattbrictson added 🐛 Bug Fix Fixes a bug automerge Automatically merge PR once all required checks pass labels Sep 7, 2024
@kodiakhq kodiakhq bot merged commit fb066d5 into main Sep 7, 2024
7 checks passed
@kodiakhq kodiakhq bot deleted the rubocop-binstub branch September 7, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatically merge PR once all required checks pass 🐛 Bug Fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant