Skip to content

fix: RELEASED_VERSION in bash script #6103

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

Merged
merged 1 commit into from
Jul 18, 2025

Conversation

hanabi1224
Copy link
Contributor

@hanabi1224 hanabi1224 commented Jul 16, 2025

Description

This PR fixes the logic of retrieving RELEASED_VERSION in ensure-version-bump-and-changelog.sh. The current logic breaks when a crate version is not tagged in the repo. e.g. in https://github.com/libp2p/rust-libp2p/actions/runs/16322151084/job/46102571787?pr=6102#step:13:35

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

@hanabi1224 hanabi1224 force-pushed the fix-released-version-in-script branch from c650e6d to 20aa231 Compare July 16, 2025 14:49
@hanabi1224 hanabi1224 marked this pull request as ready for review July 16, 2025 14:50
@jxs jxs requested a review from elenaf9 July 17, 2025 12:57
@elenaf9
Copy link
Member

elenaf9 commented Jul 17, 2025

Thanks @hanabi1224.

The CI job is right now not executed on this PR because no package was modified. Can you do a quick test where you modify some file in a crate and see if the CI job runs correctly?

@hanabi1224
Copy link
Contributor Author

@elenaf9 made some changes to libp2p/Cargo.toml and the test now fails

@elenaf9
Copy link
Member

elenaf9 commented Jul 17, 2025

@elenaf9 made some changes to libp2p/Cargo.toml and the test now fails

Okay thanks, LGTM! Please revert the change again, then we can merge.

@hanabi1224 hanabi1224 force-pushed the fix-released-version-in-script branch from 6bc97a1 to 20aa231 Compare July 18, 2025 00:03
@hanabi1224
Copy link
Contributor Author

Okay thanks, LGTM! Please revert the change again, then we can merge.

@elenaf9 Done.

@mergify mergify bot merged commit e6b0e34 into libp2p:master Jul 18, 2025
199 of 201 checks passed
@hanabi1224 hanabi1224 deleted the fix-released-version-in-script branch July 18, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants