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

[VEN-1736] Deploy ProtocolShareReserve, RiskFund, Shortfall #292

Merged
merged 7 commits into from
Sep 15, 2023
Merged

[VEN-1736] Deploy ProtocolShareReserve, RiskFund, Shortfall #292

merged 7 commits into from
Sep 15, 2023

Conversation

kkirka
Copy link
Collaborator

@kkirka kkirka commented Aug 19, 2023

This PR:

  • upgrades the version of the core pool IL depends on (to get the new SwapRouter)
  • adjusts deployment scripts for ProtocolShareReserve, RiskFund, Shortfall
  • adds the deployments to bsctestnet, bscmainnet
  • fixes a couple of small issues in the codebase (see commits)

@kkirka kkirka mentioned this pull request Aug 22, 2023
Merged
@kkirka
Copy link
Collaborator Author

kkirka commented Aug 22, 2023

CI fails because it requires VenusProtocol/venus-protocol#317. We'll need to merge 317 and change the package version to make it pass.

kkirka added a commit to VenusProtocol/venus-protocol-documentation that referenced this pull request Sep 12, 2023
@kkirka
feat: add PSR, RiskFund, Shortfall addresses
9d7f67a 
This commit adds addresses for ProtocolShareReserve, RiskFund, and Shortfall contracts. Related pull requests:

* Deployment: VenusProtocol/isolated-pools#292
* VIP: VenusProtocol/vips#59
@kkirka kkirka mentioned this pull request Sep 12, 2023
Merged
@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Health
contracts 96% 76% ✔
contracts.Lens 95% 65% ✔
contracts.Pool 100% 92% ✔
contracts.Rewards 95% 62% ✔
contracts.RiskFund 99% 71% ✔
contracts.Shortfall 100% 85% ✔
contracts.lib 100% 89% ✔
Summary 97% (1454 / 1504) 75% (442 / 590) ✔

@kkirka kkirka merged commit 3979ba1 into develop Sep 15, 2023
3 checks passed
@kkirka kkirka deleted the feat/deploy-funds branch September 15, 2023 06:31
@chechu chechu mentioned this pull request Oct 18, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chechu chechu chechu approved these changes

Assignees

@kkirka kkirka

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kkirka @chechu