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 method name in Renderer
doc [ci skip]
#27656
fix method name in Renderer
doc [ci skip]
#27656
Conversation
r? @senny (@rails-bot has picked a reviewer for you, use r? to override) |
@@ -60,7 +60,7 @@ def with_defaults(defaults) | |||
end | |||
|
|||
# Accepts a custom Rack environment to render templates in. | |||
# It will be merged with ActionController::Renderer.defaults | |||
# It will be merged with +ActionController::Renderer::DEFAULTS+ |
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.
defaults
is an attr_reader so ActionController::Renderer.defaults
will work.
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.
Ah I see what is happening, 👍
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 wonder though this statement is useful or not?
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.
May be
It will be merged with the default Rack environment defined by
ActionController::Renderer::DEFAULTS.
?
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.
Looks good. I updated. Thanks!
`ActionController::Renderer.defaults` was removed in 2db7304
f48c765
to
67471a2
Compare
Awesome, thanks 👍 |
fix method name in `Renderer` doc [ci skip]
Sure! On 5-0-stable in f1fe92e |
ActionController::Renderer.defaults
was removed in 2db7304