Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

fix: ts build ordering#42

Merged
mattsse merged 1 commit intofoundry-rs:developfrom
tynes:fix/ts-build
May 31, 2022
Merged

fix: ts build ordering#42
mattsse merged 1 commit intofoundry-rs:developfrom
tynes:fix/ts-build

Conversation

@tynes
Copy link
Copy Markdown
Collaborator

@tynes tynes commented May 30, 2022

Since packages/hardhat depends on packages/hardhat-anvil
and packages/hardhat-forge, they must be built first.
If they are not, building packages/hardhat will fail
being unable to locate the other packages.

See https://github.com/foundry-rs/hardhat/runs/6651154556?check_suite_focus=true

Error: packages/hardhat/src/index.ts(1,24): error TS2307: Cannot find module '@foundry-rs/hardhat-forge' or its corresponding type declarations.
Error: packages/hardhat/src/index.ts(2,24): error TS2307: Cannot find module '@foundry-rs/hardhat-anvil' or its corresponding type declarations.

Since `packages/hardhat` depends on `packages/hardhat-anvil`
and `packages/hardhat-forge`, they must be built first.
If they are not, building `packages/hardhat` will fail
being unable to locate the other packages.
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 30, 2022

🦋 Changeset detected

Latest commit: 784faca

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

This PR includes changesets to release 1 package
Name Type
@foundry-rs/hardhat 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

Copy link
Copy Markdown
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

ty!

@mattsse mattsse merged commit 505357b into foundry-rs:develop May 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants