-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
chore: correct typo docs #13557
base: main
Are you sure you want to change the base?
chore: correct typo docs #13557
Conversation
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.
normally we would not accept purely typo fix PRs, but changing the name of this constant seems like an improvement and I can't stop seeing the typo now. We do need all usages of the constant changed, should also make sure it builds locally
@Rjected All references have been updated, the project builds successfully. Thanks |
@@ -7,7 +7,7 @@ use alloy_primitives::{address, Address}; | |||
pub static DAO_HARDFORK_BENEFICIARY: Address = address!("bf4ed7b27f1d666546e30d74d50d173d20bca754"); | |||
|
|||
/// DAO hardfork account that ether was taken and added to beneficiary | |||
pub static DAO_HARDKFORK_ACCOUNTS: [Address; 116] = [ | |||
pub static DAO_HARDFORK_ACCOUNTS: [Address; 116] = [ |
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.
this rename has side effects. we also need to rename other occurrences of this constant
needs please don't open prs from main branch :) |
@mattsse I ran |
Description
RCP
toRPC
mesesages
tomessages
DAO_HARDKFORK_ACCOUNTS
toDAO_HARDFORK_ACCOUNTS
I fixed the typos in the comments and also one in the function. I tried really hard—happy to help, thank you!