Skip to content

chore: fix typos - #1820

Merged
seanyoung merged 3 commits into
hyperledger-solang:mainfrom
JohnWickkkk7:typos
Aug 5, 2025
Merged

chore: fix typos#1820
seanyoung merged 3 commits into
hyperledger-solang:mainfrom
JohnWickkkk7:typos

Conversation

@JohnWickkkk7

Copy link
Copy Markdown
Contributor

Description

This PR corrects several minor typos in comments and documentation across the codebase. The changes are non-functional and purely textual to improve clarity and maintain a clean, professional codebase.

Details

  • Corrected misspellings:
    • substractsubtract
    • specifedspecified
    • explicityexplicitly

These fixes help enhance readability and eliminate minor distractions during development and code reviews.

Additional Info

No logic or functionality has been modified. All changes are restricted to comments or non-executable doc annotations.

@seanyoung seanyoung left a comment

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.

Needs real name in DCO

@JohnWickkkk7

Copy link
Copy Markdown
Contributor Author

Will do, thanks!

Signed-off-by: John Wick <johnwickjonwik@gmail.com>
@JohnWickkkk7

Copy link
Copy Markdown
Contributor Author

Hi @seanyoung,
I've updated the DCO signature to use my real name as requested. Could you please review this PR again? Thanks!

Comment thread tests/imports.rs
println!("{error}");

assert!(error.contains("error: import paths 'imports', 'meh' specifed more than once"));
assert!(error.contains("error: import paths 'imports', 'meh' specified more than once"));

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.

This breaks the tests - you need to fix this elsewhere too.

Signed-off-by: John Wick <johnwickjonwik@gmail.com>
@JohnWickkkk7

Copy link
Copy Markdown
Contributor Author

Done. Please review again, thanks!

@seanyoung
seanyoung merged commit 5b33677 into hyperledger-solang:main Aug 5, 2025
15 of 17 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