-
Notifications
You must be signed in to change notification settings - Fork 46
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
blog: Add Fluentd v1.17.0 release announcement (WIP) #312
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
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.
I added quite small nitpicks. Could you read carefully and get used to be usage of ,
(comma) and enumeration of two or more items with and
?
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com> Co-authored-by: Hiroshi Hatake <hatake@calyptia.com>
Thanks! I understand! I will take care of that! |
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
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.
Also, I found errors where are newly added sections.
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com> Co-authored-by: Hiroshi Hatake <hatake@calyptia.com>
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com> Co-authored-by: Hiroshi Hatake <hatake@calyptia.com>
https://github.com/fluent/fluentd/releases/tag/v1.17.0