Skip to content

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

Merged
merged 1 commit into from
Oct 28, 2014
Merged

Conversation

ouertani
Copy link
Contributor

No description provided.

@@ -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}
Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 type

Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@viktorklang
Copy link
Contributor

@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.

@rkuhn
Copy link
Member

rkuhn commented Aug 20, 2014

LGTM

@viktorklang
Copy link
Contributor

@reactive-streams/contributors Can we merge this?

@ouertani
Copy link
Contributor Author

@reactive-streams/contributors any update please ?

@viktorklang
Copy link
Contributor

@ouertani I completely overlooked this PR and redid that work here: #127

I'm not picky about what goes in so I'll close my PR in favor of this one.

@viktorklang
Copy link
Contributor

👍 from me, @reactive-streams/contributors please vote before 24 Oct GMT+0 or this gets merged.

@viktorklang
Copy link
Contributor

@reactive-streams/contributors Merging without objections.

@viktorklang
Copy link
Contributor

@ouertani Can you rebase on top of current master?

@viktorklang viktorklang modified the milestone: 1.0.0-RC1 Oct 24, 2014
@ouertani
Copy link
Contributor Author

@viktorklang rebased on the top of current master done.

viktorklang added a commit that referenced this pull request Oct 28, 2014
@viktorklang viktorklang merged commit 02d1704 into reactive-streams:master Oct 28, 2014
@viktorklang
Copy link
Contributor

Thanks @ouertani!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants