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

chore: fix some typos #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion script/deployEIP6551.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ contract ERC6551OpenfortDeploy is Script {
bytes32 versionSalt = vm.envBytes32("VERSION_SALT");
vm.startBroadcast(deployPrivKey);

// Create an acccount to serve as implementation
// Create an account to serve as implementation
ERC6551OpenfortAccount erc6551OpenfortAccount = new ERC6551OpenfortAccount{salt: versionSalt}();

vm.stopBroadcast();
Expand Down
2 changes: 1 addition & 1 deletion script/deployManagedAccounts.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ contract ManagedOpenfortDeploy is Script, CheckOrDeployEntryPoint {
entryPoint = checkOrDeployEntryPoint();

vm.startBroadcast(deployPrivKey);
// Create an acccount to serve as implementation
// Create an account to serve as implementation
managedOpenfortAccountImpl = new ManagedOpenfortAccount{salt: versionSalt}();
// deploy account factory (beacon)
openfortFactory = new ManagedOpenfortFactory{salt: versionSalt}(
Expand Down
2 changes: 1 addition & 1 deletion script/deployUpgradeableAccounts.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ contract UpgradeableOpenfortDeploy is Script, CheckOrDeployEntryPoint {
// deploy upgradeable account implementation
vm.expectEmit(true, true, false, true);
emit AccountImplementationDeployed(CREATE2_DEPLOYER);
// Create an acccount to serve as implementation
// Create an account to serve as implementation
upgradeableOpenfortAccountImpl = new UpgradeableOpenfortAccount{salt: versionSalt}();
// deploy account factory
openfortFactory = new UpgradeableOpenfortFactory{salt: versionSalt}(
Expand Down
2 changes: 1 addition & 1 deletion script/upgradeManagedAccounts.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ contract ManagedOpenfortUpgrade is Script, CheckOrDeployEntryPoint {
console.log("Example account address: ", exampleAccountAddress);

vm.startBroadcast(deployPrivKey);
// Create an acccount to serve as new implementation
// Create an account to serve as new implementation
managedOpenfortAccountImpl = new ManagedOpenfortAccount{salt: versionSalt}();

// Update the account implementation
Expand Down
4 changes: 2 additions & 2 deletions test/foundry/core/managed/ManagedOpenfortAccountTest.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -1026,7 +1026,7 @@ contract ManagedOpenfortAccountTest is OpenfortBaseTest {
function testUpgradeBeacon() public {
address newEntryPoint = 0xDeaDbeefdEAdbeefdEadbEEFdeadbeEFdEaDbeeF;

// Check addressess
// Check addresses
assertEq(address(managedOpenfortAccountImpl.entryPoint()), address(entryPoint));

// Try to use the old and new implementation before upgrade (should always behave with current values)
Expand Down Expand Up @@ -2168,7 +2168,7 @@ contract ManagedOpenfortAccountTest is OpenfortBaseTest {
/*
* Case: User added 2 guardians and keeps the default (Openfort)
* The 2 added guardians (friends) are used to recover the account and transfer
* the ownership to beneficiary. Faild due to unsorted signatures
* the ownership to beneficiary. Failed due to unsorted signatures
* @notice Remember that signatures need to be ordered by the guardian's address.
*/
function test3GuardiansUnorderedCompleteRecovery() public {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2492,7 +2492,7 @@ contract UpgradeableOpenfortAccountTest is OpenfortBaseTest {
/*
* Case: User added 2 guardians and keeps the default (Openfort)
* The 2 added guardians (friends) are used to recover the account and transfer
* the ownership to beneficiary. Faild due to unsorted signatures
* the ownership to beneficiary. Failed due to unsorted signatures
* @notice Remember that signatures need to be ordered by the guardian's address.
*/
function test3GuardiansUnorderedCompleteRecovery() public {
Expand Down
2 changes: 1 addition & 1 deletion test/foundry/paymaster/OpenfortPaymasterV2Test.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ contract OpenfortPaymasterV2Test is OpenfortBaseTest {
vm.prank(paymasterAdmin);
openfortPaymaster.unlockStake();

// The owner trying to unlock fails because it has not passed enought time
// The owner trying to unlock fails because it has not passed enough time
vm.prank(paymasterAdmin);
vm.expectRevert();
openfortPaymaster.withdrawStake(payable(paymasterAdmin));
Expand Down