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

Token deployment on base #311

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Token deployment on base #311

wants to merge 3 commits into from

Conversation

iamsahu
Copy link
Contributor

@iamsahu iamsahu commented Jan 12, 2025

Description

This pull request introduces a new deployment script for the sepolia network in the deploy/sepolia/deploy_base_lseth.ts file. The script includes several important changes to handle the deployment and verification of the LsETH_Base contract.

Deployment script for sepolia network:

  • Added a new deployment script deploy/sepolia/deploy_base_lseth.ts that imports necessary modules and functions from hardhat and custom helpers.
  • The script defines a DeployFunction to deploy the LsETH_Base contract using the BurnMintERC20 contract and a TUPProxy for the proxy configuration.
  • Added a verification step for both the TUPProxy and BurnMintERC20 contracts after deployment.
  • Implemented a skip function to determine if the deployment should be skipped based on whether the LsETH_Base contract is already deployed.
  • Tagged the deployment function with deploy_base for easy identification and execution within the deployment process.

The deployment could be done by using the following command:

yarn hh deploy --network sepolia --deploy-scripts ./deploy/ --tags deploy_base

Notice

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you assigned this PR to yourself?
  • Have you added at least 1 reviewer?
  • Have you updated the official documentation?
  • Have you added sufficient documentation in your code?
  • Have you added relevant tests to the official test suite?

Pull Request Type

  • 💫 New Feature (Breaking Change)
  • 💫 New Feature (Non-breaking Change)
  • 🛠️ Bug fix (Non-breaking Change: Fixes an issue)
  • 🕹️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)

Testing

  • Have you tested this code with the official test suite?
  • Have you tested this code manually?

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.

1 participant