-
Notifications
You must be signed in to change notification settings - Fork 3
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
DEV: Enable some cops related to RSpec subject #32
Conversation
dd561a9
to
2bc2579
Compare
This patch enables these cops: - `RSpec/LeadingSubject` - `RSpec/EmptyLineAfterSubject` - `RSpec/NamedSubject`
2bc2579
to
a0fd05b
Compare
@@ -8,6 +8,8 @@ | |||
|
|||
let(:config) { RuboCop::Config.new } | |||
|
|||
before { config["Discourse/NoResetColumnInformationInMigrations"]["Enabled"] = true } |
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.
This cop is disabled by default (always was it seems) but for whatever reason it was working until now.
This is a followup of discussions we had on discourse/discourse#22144. To better stick to the RSpec style guide, it’s simpler to enable some of the cops related to Another PR is already done to make the necessary changes in core: discourse/discourse#22225 |
Thanks for this @Flink , nice one 🎉 |
This enables cops related to RSpec `subject`. See discourse/rubocop-discourse#32
This enables cops related to RSpec `subject`. See discourse/rubocop-discourse#32
This PR enables these cops:
RSpec/LeadingSubject
RSpec/EmptyLineAfterSubject
RSpec/NamedSubject