-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 notes for 8.16.0 #16605
Release notes for 8.16.0 #16605
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.
Left a couple a couple of suggestions if you want to consider, with the driver to help users do informed on choices after reading this.
Co-authored-by: João Duarte <[email protected]> Co-authored-by: Andrea Selva <[email protected]>
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.
Left a minimal suggestion to be more specific on where is the "if conditionals"
Co-authored-by: Andrea Selva <[email protected]>
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.
LGTM
Co-authored-by: João Duarte <[email protected]>
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.
LGTM
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.
Left some comments inline for your consideration. LMKWYT
Co-authored-by: Karen Metts <[email protected]>
Co-authored-by: Karen Metts <[email protected]>
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.
LGTM
17056 |
@logstashmachine backport 8.x |
* Update release notes for 8.16.0 * Refine release notes * Apply suggestions from code review Co-authored-by: João Duarte <[email protected]> Co-authored-by: Andrea Selva <[email protected]> * Refine release notes * Apply suggestions from code review Co-authored-by: Andrea Selva <[email protected]> * Apply suggestions from code review Co-authored-by: João Duarte <[email protected]> * Apply suggestions from code review Co-authored-by: Karen Metts <[email protected]> * Apply suggestions from code review Co-authored-by: Karen Metts <[email protected]> * Added dependency update section for new jruby version --------- Co-authored-by: logstashmachine <[email protected]> Co-authored-by: edmocosta <[email protected]> Co-authored-by: João Duarte <[email protected]> Co-authored-by: Andrea Selva <[email protected]> Co-authored-by: Karen Metts <[email protected]> Co-authored-by: Rob Bavey <[email protected]> (cherry picked from commit 8b89791)
* Update release notes for 8.16.0 * Refine release notes * Apply suggestions from code review Co-authored-by: João Duarte <[email protected]> Co-authored-by: Andrea Selva <[email protected]> * Refine release notes * Apply suggestions from code review Co-authored-by: Andrea Selva <[email protected]> * Apply suggestions from code review Co-authored-by: João Duarte <[email protected]> * Apply suggestions from code review Co-authored-by: Karen Metts <[email protected]> * Apply suggestions from code review Co-authored-by: Karen Metts <[email protected]> * Added dependency update section for new jruby version --------- Co-authored-by: logstashmachine <[email protected]> Co-authored-by: edmocosta <[email protected]> Co-authored-by: João Duarte <[email protected]> Co-authored-by: Andrea Selva <[email protected]> Co-authored-by: Karen Metts <[email protected]> Co-authored-by: Rob Bavey <[email protected]> (cherry picked from commit 8b89791) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
No description provided.