-
Notifications
You must be signed in to change notification settings - Fork 535
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
Prepare 1.0.1 #379
Prepare 1.0.1 #379
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.
Looking forward to it :)
Another thing that would be very useful to maintain here for the 1.0.1 would be a milestone to which we'd assign issues that we resolved in that version
Yeah, that's how I meant, I missed that milestones got assigned to most, thanks :) |
@ktoso Yeah, and I've just pushed a draft of release notes. Would you be able to summarize the improvements to the TCK? |
11b3358
to
f7458b9
Compare
RELEASE-NOTES.md
Outdated
+ No changes | ||
+ Improved Javadoc | ||
- TCK |
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.
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 potential resource leaks in partially consuming
Publisher
tests - Improved
Publisher
test descriptions via JavaDoc - Fixed potential resource leaks in partially emitting
Subscriber
tests - Improved
Subscriber
test descriptions via JavaDoc
5416e83
to
ff7f251
Compare
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.
Looking good, proposed some additions and links. Links to those PRs I think are helpful in case someone will have trouble updating, they could see what changed easier then
RELEASE-NOTES.md
Outdated
+ Improved JavaDoc | ||
+ Multiple test [alterations](#tck-alterations) | ||
- New contributors since 1.0.0 |
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.
weird style to be honest, usually one lists contributors who contributed to a given milestone/release, not only the new ones?
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.
Yeah, you're right, I'll list all, and then annotate the new ones :)
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.
👍
RELEASE-NOTES.md
Outdated
- Fixed potential resource leaks in partially consuming Publisher tests | ||
- Fixed potential resource leaks in partially emitting Subscriber tests | ||
- Renamed `untested_spec305_cancelMustNotSynchronouslyPerformHeavyCompuatation` to `untested_spec305_cancelMustNotSynchronouslyPerformHeavyComputation` (typo) |
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.
Link to #306
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.
👍
--- | ||
|
||
## TCK alterations |
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.
Add:
- Allow configuring separate timeout for "no events during N time", allowing for more aggressive timeouts in the rest of the test suite if required +tck #308 allow configuring "no events during N time" separately #314
- New test verifying Rule 2.10, in which subscriber must be prepared to receive onError signal without having signaled
request
before +tck #362 wait for request signal in 209, and new additional tests #374
those are more interesting than the remaining "just a fix" ones :)
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.
👍
RELEASE-NOTES.md
Outdated
|
||
- Fixed potential resource leaks in partially consuming Publisher tests | ||
- Fixed potential resource leaks in partially emitting Subscriber tests |
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.
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.
👍
RELEASE-NOTES.md
Outdated
## TCK alterations | ||
|
||
- Fixed potential resource leaks in partially consuming Publisher tests |
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.
Link to #375
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.
👍
RELEASE-NOTES.md
Outdated
- Fixed potential resource leaks in partially consuming Publisher tests (#375) | ||
- Fixed potential resource leaks in partially emitting Subscriber tests (#372, #373) | ||
- Renamed `untested_spec305_cancelMustNotSynchronouslyPerformHeavyCompuatation` to `untested_spec305_cancelMustNotSynchronouslyPerformHeavyComputation` (#306) |
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.
Have the issue numbers be links?
40a44ad
to
17cb353
Compare
@reactive-streams/contributors Please review this PR, I think we're ready to ship 1.0.1-RC1 (and have the implementations make sure that the new TCK doesn't cause any havoc). |
looks good to me |
@reactive-streams/contributors Merging this into master and tagging. Publish will happen as soon as we have someone who wants to do the honors :) |
I can probably do it later today, need to finish some other stuff first, though. |
@rkuhn That'd be much welcome. Let me know if you want to coordinate. I'm merging this to master and tagging it. |
@rkuhn merged and tagged |
Yay, thanks Roland. Ping if you'll want a sanity check of the artifact |
Manually applying aaac86e I have signed and uploaded to https://oss.sonatype.org/content/repositories/orgreactivestreams-1030, please let me know if everything arrived okay. |
@rkuhn LGTM! |
RxJava 2.1.2 snapshot passes this TCK locally (107 classes). |
Akka Streams 2.5.3 all green: Artifacts look good as well. |
|
Shipped it :-) |
@rkuhn I'ev also pushed the correct tag now: https://github.com/reactive-streams/reactive-streams-jvm/tree/v1.0.1-RC1 |
WIP