Skip to content

Improve and document how to work with detached test plugins project#9633

Merged
fab-10 merged 2 commits intobesu-eth:mainfrom
fab-10:document-detached-test-plugin-development
Jan 15, 2026
Merged

Improve and document how to work with detached test plugins project#9633
fab-10 merged 2 commits intobesu-eth:mainfrom
fab-10:document-detached-test-plugin-development

Conversation

@fab-10
Copy link
Copy Markdown
Contributor

@fab-10 fab-10 commented Jan 14, 2026

PR description

This PR is a followup to #9590, and add documentation on how to work on the detached test plugins plus some more automation

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
  • hive tests: Engine or other RPCs modified?

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
@fab-10 fab-10 force-pushed the document-detached-test-plugin-development branch from bcfef99 to 981c152 Compare January 14, 2026 13:33
@fab-10 fab-10 marked this pull request as ready for review January 14, 2026 14:00
Copilot AI review requested due to automatic review settings January 14, 2026 14:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@fab-10 fab-10 added the dev experience The build system, things that enable easier development etc. label Jan 14, 2026
Copy link
Copy Markdown
Contributor

@usmansaleem usmansaleem left a comment

Choose a reason for hiding this comment

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

LGTM. Do we need to add any additional information regarding upgrading of this gradle wrapper whenever we upgrade root project's gradle wrapper?

@fab-10
Copy link
Copy Markdown
Contributor Author

fab-10 commented Jan 15, 2026

LGTM. Do we need to add any additional information regarding upgrading of this gradle wrapper whenever we upgrade root project's gradle wrapper?

That's a good point, and it will be nice to have an automated way to warn if the gradle version is not aligned with the main project, I will think about it.

@fab-10 fab-10 enabled auto-merge (squash) January 15, 2026 10:09
@fab-10 fab-10 added the plugins label Jan 15, 2026
@github-project-automation github-project-automation bot moved this to Backlog in Plugins API Jan 15, 2026
@fab-10 fab-10 moved this from Backlog to In review in Plugins API Jan 15, 2026
@fab-10 fab-10 merged commit 8704e74 into besu-eth:main Jan 15, 2026
46 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Plugins API Jan 15, 2026
@fab-10 fab-10 deleted the document-detached-test-plugin-development branch January 15, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev experience The build system, things that enable easier development etc. plugins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants