Skip to content

Small touchups to the TCK README.md #256

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
Mar 31, 2015

Conversation

viktorklang
Copy link
Contributor

@reactive-streams/contributors I went through the TCK README.md and made some fixes/edits/typo-corrections etc. Feel free to comment, I'm intending to merge it within 24h to ensure we can ship the next RC.

Ping @ktoso

@viktorklang viktorklang self-assigned this Mar 30, 2015
@viktorklang viktorklang added this to the 1.0.0.RC4 milestone Mar 30, 2015
@@ -3,21 +3,23 @@
The purpose of the *Reactive Streams Technology Compatibility Kit* (from here on referred to as: *the TCK*) is to guide
and help Reactive Streams library implementers to validate their implementations against the rules defined in [the Specification](https://github.com/reactive-streams/reactive-streams-jvm).

The TCK is implemented using **plain Java (1.6)** and **TestNG** tests, and should be possible to use from other languages and testing libraries (such as Scala, Groovy, JRuby or others).
The TCK is implemented using **plain Java (1.6)** and **TestNG** tests, and should be possible to use from other JVM-based languages and testing libraries.
Copy link
Contributor

Choose a reason for hiding this comment

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

good idea, more inclusive.

@ktoso
Copy link
Contributor

ktoso commented Mar 30, 2015

LGTM from my side, thanks for the polish :-)

@viktorklang viktorklang force-pushed the wip-proofread-tck-readme-√ branch 2 times, most recently from 17d61ce to 7836eb0 Compare March 30, 2015 12:30
@viktorklang viktorklang force-pushed the wip-proofread-tck-readme-√ branch from 7836eb0 to 7f6a3e8 Compare March 30, 2015 12:32
@viktorklang
Copy link
Contributor Author

@ktoso I addressed all comments :)

@ktoso
Copy link
Contributor

ktoso commented Mar 30, 2015

LGTM from my side :)

viktorklang added a commit that referenced this pull request Mar 31, 2015
@viktorklang viktorklang merged commit 0d0b70e into master Mar 31, 2015
@viktorklang viktorklang deleted the wip-proofread-tck-readme-√ branch March 31, 2015 08:36
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.

2 participants