Skip to content

Deprecation warning for EVM versions older than london - #16903

Open
matheusaaguiar wants to merge 2 commits into
developfrom
deprecation_warning_evm_versions_older_london
Open

Deprecation warning for EVM versions older than london#16903
matheusaaguiar wants to merge 2 commits into
developfrom
deprecation_warning_evm_versions_older_london

Conversation

@matheusaaguiar

Copy link
Copy Markdown
Contributor

Part of #15795.
Emit a deprecation warning for EVM versions older than london.

@matheusaaguiar matheusaaguiar self-assigned this Aug 11, 2026
Comment thread test/cmdlineTests/evm_version_petersburg/input.sol Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there a reason why this test and a couple of other tests have fixed old EVM version? Would it make sense to update the EVM version (or not specify it) instead of adding the warning to the output?

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.

I don't know if there is a specific reason. But otherwise I agree that it would best to not specify it.

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.

@cameel are you aware of any special reason for fixing the EVM version in these tests?

@matheusaaguiar
matheusaaguiar force-pushed the deprecation_warning_evm_versions_older_london branch from ad01c53 to 9bdf215 Compare August 18, 2026 16:42
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