Skip to content

Repackage options#7688

Merged
jflo merged 5 commits intobesu-eth:mainfrom
jflo:repackageOptions
Oct 4, 2024
Merged

Repackage options#7688
jflo merged 5 commits intobesu-eth:mainfrom
jflo:repackageOptions

Conversation

@jflo
Copy link
Copy Markdown
Contributor

@jflo jflo commented Sep 28, 2024

Merge AFTER #7682

Updates MetricsOptions to implement CLIOptions, and to contain its unstable options.

Copy link
Copy Markdown
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.

looks pretty ok but there's some merge conflicts

jflo added 4 commits October 3, 2024 11:36
Signed-off-by: Justin Florentine <justin+github@florentine.us>
Signed-off-by: Justin Florentine <justin+github@florentine.us>
Signed-off-by: Justin Florentine <justin+github@florentine.us>
Signed-off-by: Justin Florentine <justin+github@florentine.us>
@jflo jflo force-pushed the repackageOptions branch from 8d996f2 to 35f0e73 Compare October 3, 2024 15:36
Signed-off-by: Justin Florentine <justin+github@florentine.us>
@jflo jflo requested a review from macfarla October 3, 2024 15:46
@jflo jflo enabled auto-merge (squash) October 3, 2024 16:00

/**
* Returns the plugin context.
* 2 Returns the plugin context.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
* 2 Returns the plugin context.
* Returns the plugin context.

@jflo jflo merged commit 911f12e into besu-eth:main Oct 4, 2024
2fehee pushed a commit to 2fehee/besu that referenced this pull request Oct 4, 2024
Updates MetricsOptions to implement CLIOptions, and to contain its unstable options.
---------

Signed-off-by: Justin Florentine <justin+github@florentine.us>Signed-off-by: Chulhee lee <leefehee@naver.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.

2 participants