Skip to content

blog: Add Fluentd v1.17.0 release announcement - #312

Merged
daipom merged 8 commits into
masterfrom
blog-fluentd-v1.17.0
Jun 12, 2024
Merged

blog: Add Fluentd v1.17.0 release announcement#312
daipom merged 8 commits into
masterfrom
blog-fluentd-v1.17.0

Conversation

@daipom

@daipom daipom commented Apr 30, 2024

Copy link
Copy Markdown
Contributor

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

@cosmo0920 cosmo0920 left a comment

Copy link
Copy Markdown
Contributor

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?

Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

Co-authored-by: Hiroshi Hatake <hatake@calyptia.com>
@daipom

daipom commented May 14, 2024

Copy link
Copy Markdown
Contributor Author

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?

Thanks! I understand! I will take care of that!

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

@cosmo0920 cosmo0920 left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
daipom and others added 4 commits May 16, 2024 10:05
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>
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
@daipom
daipom marked this pull request as ready for review June 6, 2024 00:00
@daipom daipom changed the title blog: Add Fluentd v1.17.0 release announcement (WIP) blog: Add Fluentd v1.17.0 release announcement Jun 6, 2024
@daipom

daipom commented Jun 6, 2024

Copy link
Copy Markdown
Contributor Author

@daipom
daipom requested a review from kenhys June 6, 2024 00:08
Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Comment thread content/blog/20240430_fluentd-v1.17.0-has-been-released.md Outdated
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Hiroshi Hatake <hatake@calyptia.com>
@daipom

daipom commented Jun 6, 2024

Copy link
Copy Markdown
Contributor Author

@cosmo0920 Thanks for your review!
I have applied them!

@cosmo0920

Copy link
Copy Markdown
Contributor

@kenhys Do you have any concerns? If there is no concerns, it's ready to :shipit:!

@daipom

daipom commented Jun 12, 2024

Copy link
Copy Markdown
Contributor Author

ping @kenhys @ashie
Please take a look when you have time 😃

@ashie ashie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@daipom
daipom merged commit c08b905 into master Jun 12, 2024
@daipom
daipom deleted the blog-fluentd-v1.17.0 branch June 12, 2024 09:20
@daipom

daipom commented Jun 12, 2024

Copy link
Copy Markdown
Contributor Author

Thanks!

@daipom

daipom commented Jun 12, 2024

Copy link
Copy Markdown
Contributor Author

@cosmo0920 @kenhys @ashie Thanks for your review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants