-
Notifications
You must be signed in to change notification settings - Fork 108
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
railties: Move the definition of PluginGenerator.app_path to railties.rbs #697
Conversation
@tk0miya Thanks for your contribution! Please follow the instructions below for each change. Available commandsYou can use the following commands by commenting on this PR.
|
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.
I'm not sure why you added a class other than railtie to patch.rbs.
ah, sorry. It's mistake... I'd like to do only moving |
7e1b663
to
fef1771
Compare
Now I busted zombies from #692. |
gems/railties/6.0/railties.rbs
Outdated
# It is necessary to satisfy alias target. | ||
# TODO: Define this method to correct place. |
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 we remove this comment?
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.
omg...
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.
removed.
….rbs As a preparation of ruby#693, this moves the definition of `Rails::Generator::PluginGenerator#app_path` to railties.rbs
fef1771
to
6aa1df0
Compare
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.
APPROVE
/merge |
As a preparation of #693, this moves the definition of
Rails::Generator::PluginGenerator#app_path
to railties.rbs