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

improve: [N04] Typographical errors #258

Merged
merged 2 commits into from
Mar 17, 2023
Merged

improve: [N04] Typographical errors #258

merged 2 commits into from
Mar 17, 2023

Conversation

nicholaspai
Copy link
Member

From Audit

Consider correcting the following typographical errors:
In BondToken.sol :
Line 11, "rootBundleProposer()" should be "rootBundleProposal()".
Line 21: "permissiong" should be "permissioning".
In EIP712CrossChainUpgradeable.sol :
Line 83: "its always" should be "it's always".
In MultiCallerUpgradeable.sol :
Line 5: "@title MockSpokePool" should be "@title MultiCallerUpgradeable".
In Optimism_Adapter.sol :
Line 23: "its only" should be "it's only".
In Polygon_SpokePool.sol :
Line 51: the sentence "See" is incomplete.
Line 139: the sentence is malformed.
In SpokePool.sol :
Line 567: "speedUpRelay()" should be "speedUpDeposit()".
Line 1177: "its always" should be "it's always".
In Succinct_Adapter.sol :
Line 27: "destinatipn" should be "destination".
Line 27: "the message.." should be "the message.".
In Succinct_SpokePool.sol :
Line 22: "callValidated" should be "adminCallValidated".
Line 24: "callValidated" should be "adminCallValidated".
Line 35: "callValidated" should be "adminCallValidated".

## From Audit

Consider correcting the following typographical errors:
In BondToken.sol :
Line 11, "rootBundleProposer()" should be "rootBundleProposal()".
Line 21: "permissiong" should be "permissioning".
In EIP712CrossChainUpgradeable.sol :
Line 83: "its always" should be "it's always".
In MultiCallerUpgradeable.sol :
Line 5: "@title MockSpokePool" should be "@title MultiCallerUpgradeable".
In Optimism_Adapter.sol :
Line 23: "its only" should be "it's only".
In Polygon_SpokePool.sol :
Line 51: the sentence "See" is incomplete.
Line 139: the sentence is malformed.
In SpokePool.sol :
Line 567: "speedUpRelay()" should be "speedUpDeposit()".
Line 1177: "its always" should be "it's always".
In Succinct_Adapter.sol :
Line 27: "destinatipn" should be "destination".
Line 27: "the message.." should be "the message.".
In Succinct_SpokePool.sol :
Line 22: "callValidated" should be "adminCallValidated".
Line 24: "callValidated" should be "adminCallValidated".
Line 35: "callValidated" should be "adminCallValidated".
@nicholaspai nicholaspai added the OZ Audit - March 2023 Resolves issue discovered in March 2023 OZ Audit label Mar 15, 2023
Copy link
Contributor

@mrice32 mrice32 left a comment

Choose a reason for hiding this comment

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

LGTM!

@nicholaspai nicholaspai merged commit 43a0ea7 into master Mar 17, 2023
@nicholaspai nicholaspai deleted the audit2/n04 branch March 17, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OZ Audit - March 2023 Resolves issue discovered in March 2023 OZ Audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants