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
Partial compliance with doc/method_documentation.rdoc #3431
Conversation
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.
Thanks for the working on this! The vast majority of the changes look good, I only found three minor issues.
array.c
Outdated
@@ -4916,19 +4767,13 @@ rb_ary_zip(int argc, VALUE *argv, VALUE ary) | |||
* | |||
* Transposes the rows and columns in an array of arrays. | |||
* | |||
* Each element in +self+ must be an | |||
* {Array-convertible object}[doc/implicit_conversion_rdoc.html#label-Array-Convertible+Objects]. | |||
* Each element in +self+ must be an \Integer. |
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.
Shouldn't this be "must be an \Array"?
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.
Fixed.
array.c
Outdated
* - Argument +string_separator+, if given, myst be a | ||
* {String-convertible object}[doc/implicit_conversion_rdoc.html#label-String-Convertible+Objects]. | ||
* - Argument +n+, if given, must be an \Integer. | ||
* - Argument +string_separator+, if given, myst be a \String. |
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.
Typo myst
-> must
.
* [] <=> CSV::Row.new([], []) # => 0 | ||
* | ||
* Returns +nil+ if +other_array+ is not an Array-convertible object: | ||
* Returns +nil+ if +other_array+ is not an \Array. | ||
* [] <=> 0 # => nil |
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.
We may want to delete this example, since it doesn't add new information.
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.
We shall delete this example, but later, in the course of eliminating low-information 1-line examples.
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.
Changes look good. Thanks!
Removes references to *-convertible thingies. See https://github.com/ruby/ruby/blob/master/doc/method_documentation.rdoc#argument-description-if-necessary-
I've done these broadside (one issue in all methods) rather than method-wise (all issues in some methods) b/c it's both easier to make the mods and easier to review.
(But should have done this in two or three pieces. Better next time.)