-
Notifications
You must be signed in to change notification settings - Fork 3
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
Script: map anima token and refactor factory scripts #49
Conversation
script/factories/mainchain/factory-maptoken-mainchain-sepolia.s.sol
Outdated
Show resolved
Hide resolved
script/factories/mainchain/factory-maptoken-mainchain-sepolia.s.sol
Outdated
Show resolved
Hide resolved
script/factories/mainchain/factory-maptoken-mainchain-ethereum.s.sol
Outdated
Show resolved
Hide resolved
811fd23
to
bd61967
Compare
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.
minor fix then merge
function run() public virtual; | ||
function _initCaller() internal virtual returns (address); | ||
function _initTokenList() internal virtual returns (uint256 totalToken, MapTokenInfo[] memory infos); | ||
|
||
function _proposeAndExecute(Proposal.ProposalDetail memory proposal) internal { | ||
proposal.executor = _governors[0]; | ||
function _propose(Proposal.ProposalDetail memory proposal) internal virtual { |
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.
function _propose(Proposal.ProposalDetail memory proposal) internal virtual { | |
function _propose(Proposal.ProposalDetail memory proposal) internal { |
); | ||
} | ||
|
||
function _proposeAndExecuteProposal(Proposal.ProposalDetail memory proposal) internal { |
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.
bring to upper
_governorPKs.push(governorsPksM[i]); | ||
for (uint256 i; i < mGovernors.length; ++i) { | ||
_governors.push(mGovernors[i]); | ||
_governorPKs.push(mGovernorsPk[i]); |
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.
create another PR to avoid load private keys
Description
This PR adds map anima token scripts and refactors factory scripts
Checklist