Skip to content

Conversation

@fjtirado
Copy link
Contributor

@fjtirado fjtirado commented Jan 31, 2025

Adds a new blog post for the Java SDK v7.0.0 release

@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch from bbf1981 to f30d4a4 Compare January 31, 2025 12:42
@fjtirado fjtirado changed the title Added release blog Adding release blog Jan 31, 2025
@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch 6 times, most recently from f5f68b9 to 8f39f94 Compare January 31, 2025 12:55
@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch from 8f39f94 to 8c29738 Compare January 31, 2025 13:00
@fjtirado fjtirado requested a review from JBBianchi January 31, 2025 13:01
@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch from 8c29738 to 3635a69 Compare January 31, 2025 13:02
Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

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

Please fix the title and description: Adding release blog is not a proper PR title:

  • it is not ubiquitous: what is released?
  • it is incorrect: you are not adding a blog, but a blog post
  • the tense is incorrect and goes against GH recommendations. It should use imperative tense

@fjtirado fjtirado changed the title Adding release blog Releasing Java SDK 7.0.0 Jan 31, 2025
@fjtirado fjtirado changed the title Releasing Java SDK 7.0.0 Release Java SDK 7.0.0 Jan 31, 2025
@fjtirado fjtirado changed the title Release Java SDK 7.0.0 Release Java SDK 7.0.0 Blog Post Jan 31, 2025
@fjtirado fjtirado requested a review from cdavernas January 31, 2025 13:26
Signed-off-by: Francisco Javier Tirado Sarti <[email protected]>
@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch from 3635a69 to 74c267b Compare January 31, 2025 13:28
@fjtirado
Copy link
Contributor Author

Please fix the title and description: Adding release blog is not a proper PR title:

* it is not ubiquitous: what is released?

* it is incorrect: you are not adding a blog, but a blog post

* the tense is incorrect and goes against GH recommendations. It should use imperative tense

To be honest I do not even check it :).... changed

Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

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

LGTM! Cheers ❤️

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

LGTM! Just a few nitpick suggestions

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini
<[email protected]>
Update src/content/blog/releases/release-sdk-java-7.0.0.md

Co-authored-by: Ricardo Zanini <[email protected]>
Signed-off-by: Francisco Javier Tirado Sarti <[email protected]>
@fjtirado fjtirado force-pushed the Release_Java_Sdk_7.0.0 branch from c85c4ff to f04ae78 Compare January 31, 2025 21:01
@ricardozanini ricardozanini merged commit 52813e3 into serverlessworkflow:main Feb 3, 2025
5 checks passed
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.

4 participants