Skip to content

Conversation

@fafk
Copy link

@fafk fafk commented Oct 13, 2025

Description

Should be merged after #249.

Artifacts from deployment of core contracts on Plasma. All contracts are verified.

Official balancer contract is not deployed on the chain, so there are no balancer-related deployment files.

@fafk fafk marked this pull request as ready for review October 13, 2025 16:04
@fafk fafk requested a review from a team as a code owner October 13, 2025 16:04
@fafk fafk changed the base branch from main to deploy-linea October 13, 2025 19:22
## Description

fix some minor issues in the comments

Signed-off-by: pennylees <techdashen@msn.com>
@fafk fafk changed the base branch from deploy-linea to main October 29, 2025 16:27
@kaze-cow kaze-cow changed the base branch from main to deploy-linea November 10, 2025 14:08
Copy link
Contributor

@kaze-cow kaze-cow left a comment

Choose a reason for hiding this comment

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

all contracts are deployed with correct bytecodes and verified. seems good to me 👍

quite a few unrelated typographical/non impact changes that would be best left in a separate PR.

break;
default:
throw new Error("unreacheable");
throw new Error("unreachable");
Copy link
Contributor

Choose a reason for hiding this comment

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

unrelated change

},
namedAccounts: {
// Note: accounts defined by a number refer to the the accounts as configured
// Note: accounts defined by a number refer to the accounts as configured
Copy link
Contributor

Choose a reason for hiding this comment

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

unrelated change

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.

4 participants