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

Standardize Variable and Function Naming in mempool and network modules #15865

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

vipocenka
Copy link

This PR updates variable and function names in coordinator.rs and dummy.rs to improve consistency and readability in the codebase. These changes align with standard naming conventions and reduce potential confusion.

Changes:

  • Updated num_committed_txns_recieved_since_peers_updatednum_committed_txns_received_since_peers_updated in coordinator.rs
  • Standardized rpc_protoclsrpc_protocols in dummy.rs

Tests

Since no functional changes were introduced, existing tests should sufficiently cover the modifications.

Copy link

trunk-io bot commented Feb 1, 2025

⏱️ 5s total CI duration on this PR

Job Cumulative Duration Recent Runs
permission-check 3s 🟥
permission-check 2s 🟥

settingsfeedbackdocs ⋅ learn more about trunk.io

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.

1 participant