7 bug release workflow failed#527
Conversation
…rgon789/foundry into 7-bug-release-workflow-failed
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
…rgon789/foundry into 7-bug-release-workflow-failed
Updated benchmark results for Foundry versions and system information. foundry-rs@1c4d334 Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
| console_sol, | ||
| ) | ||
| raw_sigs = [s.strip().strip('"') for s in sig_strings] | ||
| sigs = [ | ||
| s.replace("string", "string memory").replace("bytes)", "bytes memory)") | ||
| re.sub(r"(\w+\[\])", r"\1 memory", s) |
|
Deployment failed with the following error: Learn More: https://vercel.com/dargon789-forge?upgradeToPro=build-rate-limit |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
There was a problem hiding this comment.
Code Review
This pull request introduces several significant updates, including the addition of Tempo-specific wallet and virtual address registry commands, enhancements to the Anvil backend for multi-endpoint fork support, and improvements to the cast CLI. The review identified a critical issue where call scopes were silently ignored on pre-T3 hardforks, a potential division-by-zero error in the nightly benchmark regression script, and a race condition in the virtual address log watcher. The suggested improvements for these issues have been incorporated into the feedback.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Motivation
Solution
PR Checklist