Skip to content

docs: Fix typos and improve documentation clarity in serde-related modules #2475

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

Merged
merged 4 commits into from
May 18, 2025

Conversation

MozirDmitriy
Copy link
Contributor

Description:

This pull request addresses minor documentation and test name improvements across several consensus modules to enhance clarity and correctness:

Fixed multiple typos such as "compatibl" → "compatible", "evelope" → "envelope", and "whcih" → "which".

Updated a test function name to accurately reflect the struct it tests: test_receipt_evelope_bincode_roundtrip → test_receipt_envelope_bincode_roundtrip.

These changes are purely cosmetic and do not affect runtime behavior or logic.

This PR improves code readability and documentation accuracy, making the codebase more maintainable.

@github-project-automation github-project-automation bot moved this to Reviewed in Alloy May 18, 2025
@mattsse mattsse changed the title Fix typos and improve documentation clarity in serde-related modules docs: Fix typos and improve documentation clarity in serde-related modules May 18, 2025
@mattsse mattsse enabled auto-merge (squash) May 18, 2025 13:12
@mattsse mattsse merged commit 66d614c into alloy-rs:main May 18, 2025
27 checks passed
@github-project-automation github-project-automation bot moved this from Reviewed to Done in Alloy May 18, 2025
jgomez360 pushed a commit to jgomez360/alloy that referenced this pull request Jun 15, 2025
…dules (alloy-rs#2475)

* Update header.rs

* Update envelope.rs

* Update mod.rs

* Update envelope.rs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants