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
Deprecate EnforceSuperclass
module`
#9087
Deprecate EnforceSuperclass
module`
#9087
Conversation
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.
@@ -0,0 +1 @@ | |||
* [#9087](https://github.com/rubocop-hq/rubocop/pull/9087): Deprecate `EnforceSuperclass` module. ([@koic][]) |
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.
Any reason for not using the main changelog?
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.
RuboCop core introduces new option to prevent CHANGELOG conflicts. Please see to the bottom of PR checklist for details :-)
Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
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.
Ohh I missed that. Thanks!
Sounds good. I think we should create some RuboCop 2.0 ticket/milestone where we are tracking all changes that we have to do before cutting 2.0, so we won't forget about something. |
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.
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.
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.
Before submitting the PR make sure the following are checked:
[Fix #issue-number]
(if the related issue exists).master
(if not - rebase it).bundle exec rake default
. It executes all tests and runs RuboCop on its own code.{change_type}_{change_description}.md
if the new code introduces user-observable changes. See changelog entry format for details.