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

Use 'Rails.root.join' in rails_helper #2640

Merged
merged 3 commits into from Nov 22, 2022

Conversation

masato-bkn
Copy link
Contributor

I updated rubocop-rails gem to v2.17.3 in my project recently.
By this update, Rubocop suggests to use Rails.root.join instead of string interpolation for Rails/FilePath at 'rails_helper.rb'.

It might be better to use Rails.root.join than string interpolation too in rails_helper generator.

How do you think about this?

Copy link
Member

@pirj pirj left a comment

Choose a reason for hiding this comment

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

Thanks!

@pirj pirj marked this pull request as ready for review November 22, 2022 09:25
masato-bkn and others added 2 commits November 22, 2022 18:37
Co-authored-by: Phil Pirozhkov <pirj@users.noreply.github.com>
Co-authored-by: Phil Pirozhkov <pirj@users.noreply.github.com>
@pirj pirj merged commit 0dcce14 into rspec:main Nov 22, 2022
JonRowe added a commit that referenced this pull request Nov 22, 2022
@JonRowe
Copy link
Member

JonRowe commented Nov 21, 2023

Released in 6.1.0

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

3 participants