-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
[Fix #184] Fix Rake/Environment to allow task with no block #185
Conversation
|
||
it 'does not register an offense to task with no block' do | ||
expect_no_offenses(<<~RUBY) | ||
task(:spec).clear |
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 is a nitpick. do_something
will be a more generic naming than clear
.
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.
Thanks! I updated to task(:spec).clear
to task(:foo).do_something
Can you add the change log entry? |
@koic I added the changelog entry. |
CHANGELOG.md
Outdated
@@ -2,6 +2,10 @@ | |||
|
|||
## master (unreleased) | |||
|
|||
### Bug fixes | |||
|
|||
* [#184](https://github.com/rubocop-hq/rubocop-rails/issues/184): Fix Rake/Environment to allow task with no block. ([@hanachin][]) |
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.
Can you enclose the cop name in backticks?
-* [#184](https://github.com/rubocop-hq/rubocop-rails/issues/184): Fix Rake/Environment to allow task with no block. ([@hanachin][])
+* [#184](https://github.com/rubocop-hq/rubocop-rails/issues/184): Fix `Rake/Environment` to allow task with no block. ([@hanachin][])
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.
Yes I can 💪
Thanks! |
Fixes #184 .
Before submitting the PR make sure the following are checked:
[Fix #issue-number]
(if the related issue exists).master
(if not - rebase it).and description in grammatically correct, complete sentences.
bundle exec rake default
. It executes all tests and RuboCop for itself, and generates the documentation.