Skip to content

Conversation

@0xRVE
Copy link
Contributor

@0xRVE 0xRVE commented Jan 22, 2026

bump resolc version to 0.6.0 in github workflows

@0xRVE 0xRVE requested review from a team as code owners January 22, 2026 10:35
@0xRVE 0xRVE added the T7-smart_contracts This PR/Issue is related to smart contracts. label Jan 22, 2026
@0xRVE
Copy link
Contributor Author

0xRVE commented Jan 22, 2026

/cmd prdoc --audience runtime_dev --bump patch


- name: Install resolc
run: |
VERSION="0.3.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@xermicus @pgherveou
Why was this still 0.3.0?

@0xRVE 0xRVE requested review from pgherveou and xermicus January 22, 2026 11:07
@0xOmarA
Copy link
Contributor

0xOmarA commented Jan 22, 2026

@0xRVE The differential tests CI for PolkaVM fail since the expectations file we have in the repo says that X set tests should fail but with this version of the compiler Y set of tests now fail.

To make CI pass you should check the CI job, find the expectations file for PolkaVM, then update the one used in the repo.

You can find the expectation file in the workflow summary here: https://github.com/paritytech/polkadot-sdk/actions/runs/21245356379?pr=10873

cargo-command: "forklift cargo"
revive-differential-tests-ref: "be448d78bd83a755fdcab96849971ecdf4eebce0"
resolc-version: "0.5.0"
resolc-version: "0.6.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need that here at all for evm?

Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need it for the EVM, but it makes the action simpler

Copy link
Member

Choose a reason for hiding this comment

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

After a discussion else-where, the test suite seems to assert on hardcoded expectations or values collected from an outdated version of the pallet and compiler.

Copy link
Contributor

Choose a reason for hiding this comment

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

@xermicus Rob updated it in this commit: ed39632

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T7-smart_contracts This PR/Issue is related to smart contracts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants