Skip to content

chore(apollo_l1_endpoint_monitor): add client with minimal boilerplate #6329

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

Open
wants to merge 1 commit into
base: gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test
Choose a base branch
from

Conversation

giladchase
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@giladchase giladchase marked this pull request as ready for review May 5, 2025 12:29
Copy link
Contributor Author

giladchase commented May 5, 2025

@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from b6b1e56 to 80195c7 Compare May 5, 2025 12:34
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_implement_the_core_of_l1_endpoint_monitor branch from 7595c68 to 1669365 Compare May 5, 2025 12:34
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from 80195c7 to 619bf7e Compare May 5, 2025 12:35
@giladchase giladchase requested a review from elintul May 5, 2025 12:35
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from 619bf7e to 91b4b80 Compare May 5, 2025 12:56
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_implement_the_core_of_l1_endpoint_monitor branch from 1669365 to f1a50b9 Compare May 5, 2025 12:56
Copy link

github-actions bot commented May 5, 2025

Benchmark movements:
full_committer_flow performance improved 😺
full_committer_flow time: [30.382 ms 30.425 ms 30.470 ms]
change: [-1.6232% -1.3721% -1.1255%] (p = 0.00 < 0.05)
Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
1 (1.00%) high mild

@giladchase giladchase changed the base branch from gilad/05-05-chore_apollo_l1_endpoint_monitor_implement_the_core_of_l1_endpoint_monitor to graphite-base/6329 May 6, 2025 07:53
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from 91b4b80 to f52378a Compare May 6, 2025 07:53
@giladchase giladchase changed the base branch from graphite-base/6329 to gilad/05-06-chore_apollo_l1_endpoint_monitor_extract_rpc_call_to_const_and_add_tests May 6, 2025 07:53
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from f52378a to fbb3102 Compare May 6, 2025 08:07
@giladchase giladchase force-pushed the gilad/05-06-chore_apollo_l1_endpoint_monitor_extract_rpc_call_to_const_and_add_tests branch from c04f4cb to e6ff66d Compare May 6, 2025 08:07
@giladchase giladchase changed the base branch from gilad/05-06-chore_apollo_l1_endpoint_monitor_extract_rpc_call_to_const_and_add_tests to graphite-base/6329 May 6, 2025 08:47
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from fbb3102 to f200edc Compare May 6, 2025 08:47
@giladchase giladchase changed the base branch from graphite-base/6329 to gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test May 6, 2025 08:48
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from f200edc to 123a599 Compare May 6, 2025 08:49
@giladchase giladchase force-pushed the gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test branch from 04aeaeb to 5a27941 Compare May 6, 2025 08:49
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from 123a599 to d8c1ec5 Compare May 6, 2025 08:50
@giladchase giladchase force-pushed the gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test branch from 5a27941 to 645609e Compare May 6, 2025 08:50
@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from d8c1ec5 to d154da2 Compare May 6, 2025 08:57
@giladchase giladchase force-pushed the gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test branch from 645609e to c92792f Compare May 6, 2025 08:57
Copy link
Collaborator

@elintul elintul left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @giladchase)

@giladchase giladchase force-pushed the gilad/05-05-chore_apollo_l1_endpoint_monitor_add_client_with_minimal_boilerplate branch from d154da2 to 6917e83 Compare May 6, 2025 09:08
@giladchase giladchase force-pushed the gilad/05-06-chore_apollo_l1_endpoint_monitor_add_integration_test branch from c92792f to 5641b62 Compare May 6, 2025 09:08
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.

3 participants