Skip to content

Comments

fix: package contracts with correct paths#461

Merged
gakonst merged 2 commits intomasterfrom
fix/contracts-import-paths
Apr 14, 2021
Merged

fix: package contracts with correct paths#461
gakonst merged 2 commits intomasterfrom
fix/contracts-import-paths

Conversation

@gakonst
Copy link
Contributor

@gakonst gakonst commented Apr 14, 2021

  • Allows importing OVM contract as:
import "@eth-optimism/contracts/OVM/accounts/OVM_ECDSAContractAccount.sol";
import "@eth-optimism/contracts/iOVM/...";
import "@eth-optimism/contracts/libraries/...";
  • Removes the copyfiles build steps, these were not required as we can package artifacts and artifacts-ovm via the files field in package.json
  • Adds a prepublish step which copies over contracts/optimistic-ethereum/** to the top level directory

Fixes #458

@changeset-bot
Copy link

changeset-bot bot commented Apr 14, 2021

🦋 Changeset detected

Latest commit: 4a4c263

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/contracts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@gakonst gakonst force-pushed the fix/contracts-import-paths branch from a08f136 to 4ad830e Compare April 14, 2021 08:24
@gakonst gakonst merged commit 6626d99 into master Apr 14, 2021
@gakonst gakonst deleted the fix/contracts-import-paths branch April 14, 2021 08:38
InoMurko referenced this pull request in omgnetwork/optimism May 25, 2021
* fix: package contracts with correct paths

* chore: add changeset
theochap pushed a commit that referenced this pull request Dec 10, 2025
* chore(bin): Remove `kt`

* lint

* lint
theochap pushed a commit that referenced this pull request Dec 10, 2025
* fix: downgrade for release plz (#458)

* chore(bin): Remove `kt` (#461)

* chore(bin): Remove `kt`

* lint

* lint

* fix(client): walkback channel timeout

* update tests

* lint

---------

Co-authored-by: clabby <ben@clab.by>
theochap pushed a commit that referenced this pull request Jan 15, 2026
No need for method `OpExecutionPayloadV4::withdrawals` and the types it
imports, since the inner payload is in a publicly accessible field
anyways. This list of withdrawals should always be empty.
emhane pushed a commit that referenced this pull request Feb 4, 2026
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.

contracts README shows incorrect import path.

1 participant