8000 Maticx refactor by manoj9april · Pull Request #796 · compound-finance/comet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Conversation

manoj9april
Copy link
@manoj9april manoj9april commented Jul 24, 2023
  • removed permissions from run-scenarios.yaml
  • minor refactoring

Comment on lines 99 to 102
async enacted(deploymentManager: DeploymentManager): Promise<boolean> {
return true;
},

Copy link
Contributor

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.

Copy link
Author

Choose a reason for hiding this comment

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

cool will remove

@kevincheng96
Copy link
Contributor

Can you remove the unrelated commits?

Copy link
Contributor
@kevincheng96 kevincheng96 left a comment

Choose a reason for hiding this comment

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

LGTM

@kevincheng96 kevincheng96 merged commit 7a41030 into compound-finance:manoj9april/add-maticx Jul 25, 2023
kevincheng96 added a commit that referenced this pull request Aug 24, 2023
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0