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.

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

Gnosis deployment #2136

Merged
merged 17 commits into from
Dec 12, 2022
Merged

Gnosis deployment #2136

merged 17 commits into from
Dec 12, 2022

Conversation

jubeira
Copy link
Contributor

@jubeira jubeira commented Dec 12, 2022

Description

Deployment txs and outputs for missing tasks in Gnosis network.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Dependency changes
  • Code refactor / cleanup
  • Documentation or wording changes
  • Other

Checklist:

  • The diff is legible and has no extraneous changes
  • N/A Complex code has been commented, including external interfaces
  • N/A Tests are included for all code paths
  • The base branch is either master, or there's a description of how to merge

Issue Resolution

N/A

Copy link
Contributor

@nventuro nventuro left a comment

Choose a reason for hiding this comment

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

Looking good! You're missing the action ids however 🙃

@jubeira
Copy link
Contributor Author

jubeira commented Dec 12, 2022

Thanks -- yes, I wanted to get the list of deployments right first.

What should we do regarding contract verification of tasks that didn't include a mock pool as part of the deployment (e.g. weighted pool)? Should we 'backport' mock pool deployments to those tasks? WDYT?

@jubeira jubeira requested a review from nventuro December 12, 2022 20:10
@nventuro
Copy link
Contributor

What should we do regarding contract verification of tasks that didn't include a mock pool as part of the deployment (e.g. weighted pool)? Should we 'backport' mock pool deployments to those tasks? WDYT?

We could do this, but the effort involved is non-trivial. I'd do nothing for now and manually verify them when (if) the time comes.

@nventuro nventuro merged commit d926298 into master Dec 12, 2022
@nventuro nventuro deleted the gnosis-deployment branch December 12, 2022 20:51
@jubeira jubeira mentioned this pull request Dec 13, 2022
9 tasks
TomAFrench added a commit that referenced this pull request Jan 3, 2023
* master:
  Update deployments changelog. (#2139)
  Update address lookup (#2140)
  refactor: use SingletonAuthentication instead of Authentication (#2129)
  ProtocolFeeSplitter reorder (#2131)
  ProtocolFeeSplitter sentinel (#2132)
  Update deployments' readmes. (#2138)
  ProtocolFeeSplitter rename treasury (#2130)
  Gnosis deployment (#2136)
  fix: remove extraneous import/dependency (#2135)
  Deploy Aave Linear Factory v3 (#2134)
  remove `20220906-composable-stable-pool` from table of active deployments (#2125)
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