Skip to content

fix(l2cm): remove vaults args and remove unused script input params #683

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
Draft
0xniha wants to merge 1 commit into
base: sc-feat/l2contracts-manager
Choose a base branch
from
Draft

fix(l2cm): remove vaults args and remove unused script input params #683

0xniha wants to merge 1 commit into from

Conversation

@0xniha
Copy link

@0xniha 0xniha commented Nov 14, 2025

Closes OPT-1321

@0xniha 0xniha requested a review from 0xiamflux November 14, 2025 19:09
@linear
Copy link

linear bot commented Nov 14, 2025

OPT-1321 Fix script: vaults deployment without args

@0xniha 0xniha self-assigned this Nov 14, 2025
@0xOneTony 0xOneTony marked this pull request as draft November 27, 2025 14:29
@0xOneTony 0xOneTony added the hold Do not merge for now label Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0xiamflux 0xiamflux Awaiting requested review from 0xiamflux

At least 1 approving review is required to merge this pull request.

Assignees

@0xniha 0xniha

Labels

hold Do not merge for now

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@0xniha @0xOneTony