-
Notifications
You must be signed in to change notification settings - Fork 21
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
Release Version 1.2.0 #734
Conversation
b37f18a
to
d372cc9
Compare
f008b34
to
ccd2692
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.
Should be fine, I'm waiting for everything else to be done for a final approval though.
ccd2692
to
37ed5af
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.
The PR has to be adapted to the short notice PRs. At least #739 should be mentioned as EPIPE
is mentioned explicitly already. (There might be more noteworthy ones, I haven't checked all so far.)
37ed5af
to
018d898
Compare
@julianbrost: I rebased this PR's branch onto main. Afterwards, I reworded the CHANGELOG to group similar changes (especially everything retry-related) and include the latest additions. |
018d898
to
8206c40
Compare
8206c40
to
f0b7aa2
Compare
Looks like you have lost #692 in the process. |
This was intentional, as, on second thought, I found this change to be irrelevant for a CHANGELOG entry. |
Part of #707. Should be merged last before drafting the release.