-
Notifications
You must be signed in to change notification settings - Fork 412
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 and Spell check auto integration #3411
Conversation
|
Thanks @miles-six for the contribution! A few of these are for |
Yes,I have down it and you even can ignore some words by add to them to .codespell/wordlist.txt |
@jmrossy I have added the word 'receipient' to the word.txt file, and it will be ignored in future pull requests. |
@@ -135,7 +135,7 @@ impl MigrationTrait for Migration { | |||
"origin_tx"."{tx_egp}" AS "origin_tx_effective_gas_price", | |||
"origin_tx"."{tx_nonce}" AS "origin_tx_nonce", | |||
"origin_tx"."{tx_sender}" AS "origin_tx_sender", | |||
"origin_tx"."{tx_receipient}" AS "origin_tx_recipient", | |||
"origin_tx"."{tx_recipient}" AS "origin_tx_recipient", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to please also revert the changes to this database migration file or it will break the Hyperlane explorer
@jmrossy The overall modifications have been a bit chaotic. To avoid delaying the team's work, I will close this pull request and submit a new one with all the changes properly incorporated.! |
Description
I fixed many typos and want to integrate Spell check to workflow so can fix all typos in the futures
Drive-by changes
Related issues
Backward compatibility
Testing