-
Notifications
You must be signed in to change notification settings - Fork 534
Enhance java doc #97
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
Enhance java doc #97
Conversation
@@ -3,6 +3,8 @@ | |||
/** | |||
* A Processor represents a processing stage—which is both a {@link Subscriber} | |||
* and a {@link Publisher} and obeys the contracts of both. | |||
* @param <T> the Type of element signaled to the {@link Subscriber} |
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.
Is Type
intended to be capitalized?
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.
No, but as used in Subscriber java doc.
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.
Great that you followed the convention. What would you propose? Type or type?
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.
👍 type
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.
Deal! :)
Would you mind updating the other files?
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.
done
@ouertani, thanks for helping out! We're following https://github.com/reactive-streams/reactive-streams/blob/master/CONTRIBUTING.md for contributions, please read and adjust PR accordingly. |
LGTM |
@reactive-streams/contributors Can we merge this? |
@reactive-streams/contributors any update please ? |
👍 from me, @reactive-streams/contributors please vote before 24 Oct GMT+0 or this gets merged. |
@reactive-streams/contributors Merging without objections. |
@ouertani Can you rebase on top of current master? |
f93e158
to
f344818
Compare
@viktorklang rebased on the top of current master done. |
Thanks @ouertani! |
No description provided.