-
Notifications
You must be signed in to change notification settings - Fork 168
Maticx refactor #796
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
Maticx refactor #796
Conversation
async enacted(deploymentManager: DeploymentManager): Promise<boolean> { | ||
return true; | ||
}, | ||
|
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 shouldn't be true unless the proposal was actually created.
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.
cool will remove
Can you remove the unrelated commits? |
74d7d86
to
1fa065c
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.
LGTM
* Add MATICx as collateral against polygon usdc (#777) * initial migration script to add maticx as collateral for polygon usdc * add verify block * formatting: white spaces removal * Linting fixes (#783) * minor fix * linting fix * Clean up * maticx proposal fixes (#786) * maticx yaml permissions fix * Upgrade git action to use new seacrest (#782) * upgrade git action to use new seacrest * updated asset config values * added description * Fix liquidition bot scenarios * Fix lint * Maticx refactor (#796) * code refactor * lint fix * removed unwanted code * maticx description changes (#798) --------- Co-authored-by: Manoj Patra <manoj9april@gmail.com>
run-scenarios.yaml