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

Add VersionAdded to Style/HashTransformKeys, Style/HashTransformValues #7757

Merged
merged 1 commit into from Feb 25, 2020

Conversation

n0h0
Copy link
Contributor

@n0h0 n0h0 commented Feb 25, 2020

This PR added VersionAdded to Style/HashTransformKeys and Style/HashTransformValue.
This Cops was added at #7663 and published at 0.80.0.


Before submitting the PR make sure the following are checked:

  • Wrote good commit messages.
  • Commit message starts with [Fix #issue-number] (if the related issue exists).
  • Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • Added tests.
  • Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
  • The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
  • Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

CHANGELOG.md Outdated Show resolved Hide resolved
This PR added `VersionAdded` to `Style/HashTransformKeys` and `Style/HashTransformValue`.
This Cops was added at rubocop#7663 and published at 0.80.0.
@n0h0 n0h0 force-pushed the add-versionadded-to-hash-transform branch from 43e72e7 to 8fbe934 Compare February 25, 2020 17:04
@n0h0
Copy link
Contributor Author

n0h0 commented Feb 25, 2020

@koic
I fixed #7757 (comment). Would you please confirm it?

@koic koic merged commit 9074c6b into rubocop:master Feb 25, 2020
@koic
Copy link
Member

koic commented Feb 25, 2020

Thanks!

@bbatsov
Copy link
Collaborator

bbatsov commented Feb 26, 2020

We should add the version that the cop was added in to the pending cops warning, so it's easier to figure out which cop came from which release (I doubt all users update after each release). This will also make it easier to stop issues like this one, although eventually we should start checking for missing version in the CI.

@n0h0 n0h0 deleted the add-versionadded-to-hash-transform branch February 26, 2020 13:40
koic added a commit to koic/rubocop that referenced this pull request Feb 29, 2020
Follow up of rubocop#7757 (comment).

When warning about a pending cop, this PR displays the version with the cop added.

## Before

```console
% rubocop
The following cops were added to RuboCop, but are not configured. Please
set Enabled to either `true` or `false` in your `.rubocop.yml` file:
 - Style/HashEachMethods
 - Style/HashTransformKeys
 - Style/HashTransformValues
```

## After

```console
% rubocop
The following cops were added to RuboCop, but are not configured. Please
set Enabled to either `true` or `false` in your `.rubocop.yml` file:
 - Style/HashEachMethods (0.80)
 - Style/HashTransformKeys (0.80)
 - Style/HashTransformValues (0.80)
For more information: https://docs.rubocop.org/en/latest/versioning/
```
bbatsov pushed a commit that referenced this pull request Feb 29, 2020
Follow up of #7757 (comment).

When warning about a pending cop, this PR displays the version with the cop added.

## Before

```console
% rubocop
The following cops were added to RuboCop, but are not configured. Please
set Enabled to either `true` or `false` in your `.rubocop.yml` file:
 - Style/HashEachMethods
 - Style/HashTransformKeys
 - Style/HashTransformValues
```

## After

```console
% rubocop
The following cops were added to RuboCop, but are not configured. Please
set Enabled to either `true` or `false` in your `.rubocop.yml` file:
 - Style/HashEachMethods (0.80)
 - Style/HashTransformKeys (0.80)
 - Style/HashTransformValues (0.80)
For more information: https://docs.rubocop.org/en/latest/versioning/
```
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