Skip to content

ci: Add Foundry tooling integration test#511

Open
elle-j wants to merge 19 commits intomainfrom
lj/ci-foundry
Open

ci: Add Foundry tooling integration test#511
elle-j wants to merge 19 commits intomainfrom
lj/ci-foundry

Conversation

@elle-j
Copy link
Copy Markdown
Contributor

@elle-j elle-j commented Apr 29, 2026

Description

Adds a CI workflow that builds and tests a small Foundry project with foundry-polkadot's forge, exercising the standard JSON interface between resolc and foundry-polkadot end-to-end.

Resolved Issues

Closes #513

Copy link
Copy Markdown
Contributor Author

@elle-j elle-j May 1, 2026

Choose a reason for hiding this comment

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

MyToken.sol is from polkavm-hardhat-examples which is also used in the accompanying hardhat integration PR.

@elle-j elle-j changed the title ci: Add tooling integration tests ci: Add Foundry tooling integration test May 1, 2026
@elle-j elle-j marked this pull request as ready for review May 1, 2026 11:48
@elle-j elle-j requested review from kvpanch and xermicus May 1, 2026 11:49
Comment on lines +25 to +27
# Once a release includes the updated supported versions, this can be changed to a
# tag in order to download the prebuilt binary.
CI_FOUNDRY_POLKADOT_COMMIT: b3173d0584382687cc96b2af072d8cb2addf23d3
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They plan a new release mid-May.

Copy link
Copy Markdown
Member

@xermicus xermicus left a comment

Choose a reason for hiding this comment

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

LGTM - Same remarks as with #516

Additionally can you please add a comment somewhere that explicitly states that this only tests for compilation and E2E integration with tooling but not correct runtime behavior?

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.

CI: Implement foundry integration test

2 participants