Skip to content

Main#513

Closed
Dargon789 wants to merge 0 commit into
masterfrom
main
Closed

Main#513
Dargon789 wants to merge 0 commit into
masterfrom
main

Conversation

@Dargon789
Copy link
Copy Markdown
Owner

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry @Dargon789, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@vercel
Copy link
Copy Markdown

vercel Bot commented May 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
next Canceled Canceled May 2, 2026 5:52am

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented May 2, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors documentation generation and Chisel to use the solar AST and introduces several new linter rules, including rtlo, could-be-immutable, and missing-zero-check. It also updates foundryup for release resolution via the GitHub API and improves Tempo chain support in Anvil. Feedback suggests enhancing the new linter rules by including enums and user-defined value types in immutable checks, correctly handling reassignments and member access in address zero-check analysis, and making the foundryup script more robust against unordered JSON responses.

Comment thread crates/lint/src/sol/gas/immutable.rs
Comment thread crates/lint/src/sol/low/missing_zero_check.rs
Comment thread crates/lint/src/sol/low/missing_zero_check.rs
Comment thread foundryup/foundryup
@Dargon789 Dargon789 closed this May 2, 2026
@vercel vercel Bot temporarily deployed to Preview – react May 2, 2026 06:21 Inactive
@vercel vercel Bot temporarily deployed to Preview – next May 2, 2026 06:21 Inactive
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.

1 participant