-
-
Notifications
You must be signed in to change notification settings - Fork 268
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
Support local
as an environment for Rails/UnknownEnv
from Rails 7.1 onward.
#894
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
* [#893](https://github.com/rubocop/rubocop-rails/issues/893): Support `local` as an environment for `Rails/UnknownEnv` from Rails 7.1 onward. ([@ghiculescu][]) |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -24,6 +24,8 @@ | |||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^^^^ Unknown environment `developpment`. Did you mean `development`? | ||||||||||||||||||||||||||||||||||||||||||||
Rails.env.something? | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^ Unknown environment `something`. | ||||||||||||||||||||||||||||||||||||||||||||
Rails.env.local? | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^ Unknown environment `local`. | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
|
@@ -36,6 +38,9 @@ | |||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
'something' === Rails.env | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^^ Unknown environment `something`. | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
'local' === Rails.env | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^ Unknown environment `local`. | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -49,6 +54,9 @@ | |||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^^^^ Unknown environment `developpment`. | ||||||||||||||||||||||||||||||||||||||||||||
Rails.env.something? | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^ Unknown environment `something`. | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
Rails.env.local? | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^ Unknown environment `local`. | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
|
@@ -61,6 +69,9 @@ | |||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
'something' === Rails.env | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^^^^^ Unknown environment `something`. | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
'local' === Rails.env | ||||||||||||||||||||||||||||||||||||||||||||
^^^^^^^ Unknown environment `local`. | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
@@ -72,4 +83,30 @@ | |||||||||||||||||||||||||||||||||||||||||||
Rails.env == 'production' | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
context 'Rails 7.1' do | ||||||||||||||||||||||||||||||||||||||||||||
let(:config) do | ||||||||||||||||||||||||||||||||||||||||||||
RuboCop::Config.new( | ||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||
'AllCops' => { | ||||||||||||||||||||||||||||||||||||||||||||
'TargetRailsVersion' => '7.1' | ||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||
'Rails/UnknownEnv' => { | ||||||||||||||||||||||||||||||||||||||||||||
'Environments' => %w[ | ||||||||||||||||||||||||||||||||||||||||||||
development | ||||||||||||||||||||||||||||||||||||||||||||
production | ||||||||||||||||||||||||||||||||||||||||||||
test | ||||||||||||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||
it 'accepts local as an environment name on Rails 7.1' do | ||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+87
to
+105
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Now that #1134 has been merged, I think this could be changed to use the shared context and
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @ghiculescu ping. |
||||||||||||||||||||||||||||||||||||||||||||
expect_no_offenses(<<~RUBY) | ||||||||||||||||||||||||||||||||||||||||||||
Rails.env.local? | ||||||||||||||||||||||||||||||||||||||||||||
Rails.env == 'local' | ||||||||||||||||||||||||||||||||||||||||||||
RUBY | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
end | ||||||||||||||||||||||||||||||||||||||||||||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed this PR and added support for
local
as part #906 as well. I think this approach is better, although it may make sense to cherry pick my commit introducing a:rails71
shared context (or I can open a separate PR for that, but it seems small) rather than a custom config, especially given the rest of this config is just the defaults.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've extracted the shared context into its own PR in #1134.