Skip to content

Conversation

cctdaniel
Copy link
Contributor

@cctdaniel cctdaniel commented Mar 4, 2025

Summary

Added deployment script for Pulse contracts on EVM chains with configurable initialization parameters. This includes:

  • New deploy_evm_pulse_contracts.ts script for deploying PulseUpgradeable and ERC1967Proxy
  • Added EvmPulseContract class for interacting with deployed Pulse contracts
  • Fixed bytecode handling in deployIfNotCached to support different bytecode formats

Rationale

These changes are necessary to:

  1. Support deployment of Pulse contracts across different EVM chains
  2. Allow customization of contract initialization parameters via CLI
  3. Fix bytecode handling issues with different compiler outputs (object vs string)
  4. Provide a consistent interface for interacting with Pulse contracts after deployment

How has this been tested?

  • Current tests cover my changes
  • Manually tested the code

I've manually tested the deployment script on multiple EVM chains (Linea Sepolia, Ethereum Sepolia) with different initialization parameters. The script now correctly:

  • Handles bytecode in different formats
  • Loads the ERC1967Proxy artifact from the correct location
  • Allows customization of the default provider and other initialization parameters
  • Properly saves the deployed contract to the store

Copy link

vercel bot commented Mar 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 6, 2025 3:23am
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 6, 2025 3:23am
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 6, 2025 3:23am
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
component-library ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 3:23am
entropy-debugger ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 3:23am
insights ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 3:23am

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