Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: dedup error messages #9481

Merged
merged 1 commit into from
Dec 4, 2024
Merged

feat: dedup error messages #9481

merged 1 commit into from
Dec 4, 2024

Conversation

DaniPopes
Copy link
Member

@DaniPopes DaniPopes commented Dec 4, 2024

A very common pattern in errors is displaying some message: {inner}, so when chaining we end up with some message: {inner}; {inner}....

- backend: failed while inspecting; transaction validation error: call gas cost exceeds the gas limit; call gas cost exceeds the gas limit; 
+ EVM error; transaction validation error: call gas cost exceeds the gas limit

@DaniPopes DaniPopes force-pushed the dani/dedup-evm-errors branch from cb1639f to 96c738b Compare December 4, 2024 10:43
@DaniPopes DaniPopes merged commit 8ac30d9 into master Dec 4, 2024
21 checks passed
@DaniPopes DaniPopes deleted the dani/dedup-evm-errors branch December 4, 2024 11:08
@grandizzy grandizzy added T-feature Type: feature C-forge Command: forge labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge T-feature Type: feature
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

4 participants