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.

Sign up for GitHub

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

Jump to bottom

Removing payable #817

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Removing payable #817

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
4 changes: 2 additions & 2 deletions contracts/libraries/SafeMigration.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,19 +21,19 @@
/**
* @notice Address of this contract
*/
address public immutable MIGRATION_SINGLETON;

Check warning on line 24 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase

Check warning on line 24 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase
/**
* @notice Address of the Safe Singleton implementation
*/
address public immutable SAFE_SINGLETON;

Check warning on line 28 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase

Check warning on line 28 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase
/**
* @notice Address of the Safe Singleton (L2) implementation
*/
address public immutable SAFE_L2_SINGLETON;

Check warning on line 32 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase

Check warning on line 32 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase
/**
* @notice Address of the Fallback Handler
*/
address public immutable SAFE_FALLBACK_HANDLER;

Check warning on line 36 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase

Check warning on line 36 in contracts/libraries/SafeMigration.sol

View workflow job for this annotation

GitHub Actions / lint 

Variable name must be in mixedCase

/**
* @notice Event indicating a change of a singleton address. Named master copy here for legacy reasons.
Expand Down Expand Up @@ -82,7 +82,7 @@
*/
function migrateWithFallbackHandler() external onlyDelegateCall {
migrateSingleton();
ISafe(payable(address(this))).setFallbackHandler(SAFE_FALLBACK_HANDLER);
ISafe(address(this)).setFallbackHandler(SAFE_FALLBACK_HANDLER);
}

/**
Expand All @@ -99,7 +99,7 @@
*/
function migrateL2WithFallbackHandler() external onlyDelegateCall {
migrateL2Singleton();
ISafe(payable(address(this))).setFallbackHandler(SAFE_FALLBACK_HANDLER);
ISafe(address(this)).setFallbackHandler(SAFE_FALLBACK_HANDLER);
}

/**
Expand Down
Loading