Skip to content
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

Check released version and configured version #196

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

cdivitotawela
Copy link
Contributor

@cdivitotawela cdivitotawela commented Jul 29, 2024

Recently same version published and caused issues for the downstream applications with checksum changes. Added GitHub step check whether the version configured has been released previously. This will prevent overriding existing releases

fixes consensys/protocol-misc#937

Recently same version published and caused issues for the downstream applications with checksum changes. Added GitHub step check whether the version configured has been released previously. This will prevent overriding existing releases

Signed-off-by: Chaminda Divitotawela <cdivitotawela@gmail.com>
Copy link
Contributor

@macfarla macfarla left a comment

Choose a reason for hiding this comment

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

@garyschulte can you take a look please

.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Show resolved Hide resolved
cdivitotawela and others added 2 commits July 30, 2024 15:41
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Chaminda Divitotawela <cdivitotawela@users.noreply.github.com>
Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Chaminda Divitotawela <cdivitotawela@users.noreply.github.com>
@jframe jframe merged commit 5001ac8 into hyperledger:main Aug 1, 2024
11 checks passed
@garyschulte
Copy link
Contributor

This seems to have broken CI for releases. I am going to revert in the 0.9.4 release and we can come up with something else

garyschulte added a commit that referenced this pull request Aug 13, 2024
garyschulte added a commit that referenced this pull request Aug 13, 2024
This reverts commit 5001ac8.

Signed-off-by: garyschulte <garyschulte@gmail.com>
garyschulte added a commit that referenced this pull request Aug 13, 2024
This reverts commit 5001ac8.

Signed-off-by: garyschulte <garyschulte@gmail.com>
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