Skip to content

Conversation

@JackPGreen
Copy link
Contributor

@JackPGreen JackPGreen commented Nov 7, 2025

Partially automates the steps listed in the C++ Client Release Process documentation.

Specifically, steps:

  • Create branch
  • Update next version

By scripting and chaining the steps, the (manual) release work is reduced and consistency is improved.

Testing (in my fork):

Post-merge actions:

  • update docs

Partially addresses: DI-669

…tes [DI-669]

_Partially_ automates the steps listed in [the `C++ Client Release Process` documentation](https://hazelcast.atlassian.net/wiki/spaces/HZC/pages/129810774/C+Client+Release+Process).

Specifically, steps:
- `Create branch`
- `Update next version`

By scripting and chaining the steps, the (manual) release work is reduced and consistency is improved.

Testing (in my fork):
- [example execution](https://github.com/JackPGreen/hazelcast-cpp-client/actions/runs/19166496611)
- [branch created](https://github.com/JackPGreen/hazelcast-cpp-client/tree/5.5.0)
- [version update PR raised](#5)

Post-merge actions:
- [ ] update docs

_Partially addresses_: [DI-669](https://hazelcast.atlassian.net/browse/DI-669)
@JackPGreen JackPGreen requested a review from nishaatr November 7, 2025 13:44
nishaatr
nishaatr previously approved these changes Nov 7, 2025
Copy link
Contributor

@nishaatr nishaatr left a comment

Choose a reason for hiding this comment

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

Good from my side
if we keep using PR its fine but still would prefer direct push

@JackPGreen
Copy link
Contributor Author

Good from my side if we keep using PR its fine but still would prefer direct push

I'll raise on the standup next week. It's probably a wider issue than the C++ client.

git push origin ${{ steps.project_version.outputs.version }}

update-version:
name: Update version in `${{ inputs.BRANCH_NAME }}` to `${{ inputs.NEW_VERSION }}`

Choose a reason for hiding this comment

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

Suggested change
name: Update version in `${{ inputs.BRANCH_NAME }}` to `${{ inputs.NEW_VERSION }}`
name: Set next development version in `${{ inputs.BRANCH_NAME }}` to `${{ inputs.NEW_VERSION }}`

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure what the issue is with how it was?

Co-authored-by: Łukasz Dziedziul <l.dziedziul@gmail.com>
nishaatr
nishaatr previously approved these changes Nov 10, 2025
Copy link
Contributor

@nishaatr nishaatr left a comment

Choose a reason for hiding this comment

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

image

@JackPGreen
Copy link
Contributor Author

image

Might've got carried away with find and replace...

35c32ca

@JackPGreen JackPGreen requested a review from nishaatr November 12, 2025 10:12
Copy link
Contributor

@nishaatr nishaatr left a comment

Choose a reason for hiding this comment

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

forgot to mention about maintenance branch
here it says create z-branch
wondering if z-branch is actually used or how frequently! may be no need

@JackPGreen
Copy link
Contributor Author

forgot to mention about maintenance branch here it says create z-branch wondering if z-branch is actually used or how frequently! may be no need

Good point. I've added the optional ability to supply a branch name in 22533ba

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