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

fix: typos of different importance #359

Merged
merged 3 commits into from
Dec 16, 2024
Merged

Conversation

crStiv
Copy link
Contributor

@crStiv crStiv commented Dec 16, 2024

Description

This pull request addresses minor typographical fixes in the codebase to improve clarity and maintain consistency. The changes include correcting spelling errors in comments and documentation. The modifications are straightforward and do not affect functionality.

Changes

  1. scripts/check-changelog.sh:

    • Fixed typo: "explicitely" → "explicitly".
    • Adjusted wording for better clarity.
  2. src/merkle/smt/mod.rs:

    • Corrected spelling: "accomodate" → "accommodate".
    • Improved phrasing in comment: "when can remove" → "then can remove".
  3. src/merkle/store/tests.rs:

    • Fixed typo: "the the" → "the".
    • Clarified phrasing in test description.

Checklist Before Requesting Review

  • Repo forked and branch created from next according to naming convention.
  • Commit messages and code style follow conventions.
  • Relevant issues linked in the PR description (if applicable).
  • Tests updated if necessary (no functional changes, so no new tests required).
  • Documentation and comments updated to reflect changes.

Additional Notes

Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you for fixing these.

@bobbinth bobbinth merged commit 1444bbc into 0xPolygonMiden:main Dec 16, 2024
12 of 13 checks passed
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