-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
adding description of return value [ci skip] #21480
Conversation
r? @pixeltrix (@rails-bot has picked a reviewer for you, use r? to override) |
d8418c0
to
edc94b7
Compare
Hello @amitsuroliya! You are right that the sentence doesn't read well, but stating that the method returns true or false might be misinterpreted as the actual values Maybe @fxn can chime in here. We could also just say
|
"true" and "false" have a well-defined meaning in our documentation.. but I agree the "whether" version reads better. |
edc94b7
to
ddc05b6
Compare
@@ -141,7 +141,7 @@ def initialize(source, identifier, handler, details) | |||
@compile_mutex = Mutex.new | |||
end | |||
|
|||
# Returns if the underlying handler supports streaming. If so, | |||
# Returns whether the underlying handler supports streaming. If so, | |||
# a streaming buffer *may* be passed when it start rendering. |
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.
Since we're touching this anyway it would be great to expand the *may*
. I don't understand what that means.
👍 to "whether". |
adding description of return value [ci skip]
No description provided.