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

Import EnforceSuperclass module #390

Merged
merged 2 commits into from
Nov 24, 2020

Conversation

koic
Copy link
Member

@koic koic commented Nov 23, 2020

Follow rubocop/rubocop#7868.

This PR imports enforce superclass module and uses Cop::Base API for cops mixing EnforceSuperclass module.


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.
  • If this is a new cop, consider making a corresponding update to the Rails Style Guide.

@koic koic force-pushed the import_enforce_superclass_module branch from 6ba0f6b to 12c167d Compare November 23, 2020 22:34
@koic koic changed the title Import enforce superclass module Import EnforceSuperclass module Nov 23, 2020
@koic koic force-pushed the import_enforce_superclass_module branch from 12c167d to 6ec8aec Compare November 23, 2020 22:52
Follow rubocop/rubocop#7868.

This PR uses `Cop::Base` API for cops mixing `EnforceSuperclass` module.
@koic koic force-pushed the import_enforce_superclass_module branch from 6ec8aec to 52b8480 Compare November 23, 2020 22:54
koic added a commit to koic/rubocop that referenced this pull request Nov 23, 2020
RuboCop core depended on this module when it supported Rails
department. Rails department has been extracted to RuboCop Rails gem.
I forgot to remove it when extracting Rails department from the core.

rubocop/rubocop-rails#390 is ready to be removed from RuboCop core.
First of all, as soft deprecation, the document only indicates
that it is a deprecation warning.
After releasing RuboCop Rails several times, I will display a
deprecation warning and will remove it in RuboCop 2.0.
bbatsov pushed a commit to rubocop/rubocop that referenced this pull request Nov 24, 2020
RuboCop core depended on this module when it supported Rails
department. Rails department has been extracted to RuboCop Rails gem.
I forgot to remove it when extracting Rails department from the core.

rubocop/rubocop-rails#390 is ready to be removed from RuboCop core.
First of all, as soft deprecation, the document only indicates
that it is a deprecation warning.
After releasing RuboCop Rails several times, I will display a
deprecation warning and will remove it in RuboCop 2.0.
@koic koic merged commit 4890714 into rubocop:master Nov 24, 2020
@koic koic deleted the import_enforce_superclass_module branch November 24, 2020 15:58
koic added a commit to koic/rubocop that referenced this pull request Nov 17, 2021
Follow up to rubocop#9087.

This PR displays deprecation message for `EnforceSuperclass` module.

> rubocop/rubocop-rails#390 is ready to be removed from RuboCop core.
> First of all, as soft deprecation, the document only indicates that it is a deprecation warning.
> After releasing RuboCop Rails several times, I will display a deprecation warning and
> will remove it in RuboCop 2.0.

After rubocop#9087, I've released RuboCop Rails more than 10 times in about a year,
so it's time to get a deprecated warning.
bbatsov pushed a commit to rubocop/rubocop that referenced this pull request Nov 17, 2021
Follow up to #9087.

This PR displays deprecation message for `EnforceSuperclass` module.

> rubocop/rubocop-rails#390 is ready to be removed from RuboCop core.
> First of all, as soft deprecation, the document only indicates that it is a deprecation warning.
> After releasing RuboCop Rails several times, I will display a deprecation warning and
> will remove it in RuboCop 2.0.

After #9087, I've released RuboCop Rails more than 10 times in about a year,
so it's time to get a deprecated warning.
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

1 participant