Skip to content

Deployment preparation: StableSurgeHook (fixes)#162

Merged
jubeira merged 4 commits into
masterfrom
deployment-preparation/stable-surge-hook-fixes
Feb 6, 2025
Merged

Deployment preparation: StableSurgeHook (fixes)#162
jubeira merged 4 commits into
masterfrom
deployment-preparation/stable-surge-hook-fixes

Conversation

@jubeira
Copy link
Copy Markdown
Contributor

@jubeira jubeira commented Feb 6, 2025

Deployment preparation: StableSurgeHook

build-info: balancer/balancer-v3-monorepo@767a6a1 (includes fixes from audit)

Deployment preparation tasks checklist:

  • Deployments README is updated with the new task name and link
  • Task README has a description and links to the artifacts
  • build-info is updated with artifacts compiled at the specified commit
  • Artifacts are generated from build-info
  • New task has a working fork test

Deprecating old tasks checklist: N/A

Code checklist:

  • The diff is legible and has no extraneous changes
  • Complex code has been commented in tests and scripts
  • Fork test block number and test cases are appropriate for this deployment
  • The base branch is either master, or there's a description of how to merge

Issue Resolution

N/A

@jubeira jubeira requested a review from joaobrunoah February 6, 2025 16:02
joaobrunoah
joaobrunoah previously approved these changes Feb 6, 2025
Copy link
Copy Markdown
Member

@joaobrunoah joaobrunoah left a comment

Choose a reason for hiding this comment

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

LGTM! The build-info matches the specified commit. I also verified that StablePool.json did not change.
The CI failure is unrelated to this PR.

Copy link
Copy Markdown
Member

@joaobrunoah joaobrunoah left a comment

Choose a reason for hiding this comment

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

LGTM!

@jubeira jubeira merged commit 6455c84 into master Feb 6, 2025
@jubeira jubeira deleted the deployment-preparation/stable-surge-hook-fixes branch February 6, 2025 18:43
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