10000 Fix the version printed in the overview by fab-10 · Pull Request #8555 · hyperledger/besu · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix the version printed in the overview #8555

New issue
Jump to bottom
Merged
merged 2 commits into from
Apr 16, 2025
Merged

Fix the version printed in the overview #8555

merged 2 commits into from
Apr 16, 2025
+11 −2

Conversation

fab-10
Copy link
Contributor
@fab-10 fab-10 commented Apr 16, 2025 •
edited
Loading

PR description

The refactor in #8536 introduced a change in the version logged as part of the overview, instead of the short version Besu version 25.4.1 the long version is used Besu version besu/v25.4.1/linux-x86_64/openjdk-java-21, this PR restore the previous behavior.

Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests

@fab-10
Fix the version printed in the overview
f432a3b 
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
@fab-10 fab-10 force-pushed the fix-release-version-in-the-overview branch from 84f167f to f432a3b Compare April 16, 2025 17:24
@macfarla macfarla enabled auto-merge (squash) April 16, 2025 22:10
@macfarla macfarla merged commit 442e8f3 into hyperledger:main Apr 16, 2025
43 checks passed
@fab-10 fab-10 deleted the fix-release-version-in-the-overview branch April 17, 2025 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jflo jflo jflo approved these changes

@macfarla macfarla macfarla approved these changes

Assignees
No one assigned
Labels
479A
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fab-10 @jflo @macfarla
0