Skip to content

Fix/l3#8

Draft
Mohiiit wants to merge 11 commits intomainfrom
fix/l3
Draft

Fix/l3#8
Mohiiit wants to merge 11 commits intomainfrom
fix/l3

Conversation

@Mohiiit
Copy link
Copy Markdown

@Mohiiit Mohiiit commented Nov 13, 2025

Short description of what this PR does.


A longer description, include motivation and intent if possible. Detail any caveats or follow-up steps still required.


Consider motivating any new dependencies.


Delete once completed:

  • link any issues closed by this pull-request
  • add all user-facing changes to CHANGELOG.md
  • new dependencies were added to the workspace toml

Comment thread crates/pathfinder/src/bin/pathfinder/main.rs
Comment thread crates/pathfinder/src/config.rs
Comment thread crates/pathfinder/src/validator.rs Outdated
@byteZorvin byteZorvin self-assigned this Nov 14, 2025
Copy link
Copy Markdown
Member

@byteZorvin byteZorvin left a comment

Choose a reason for hiding this comment

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

it just returns false tho? can be removed?

Haha, need to punish claude

Comment thread crates/pathfinder/src/config.rs
Copy link
Copy Markdown
Author

@Mohiiit Mohiiit left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread crates/pathfinder/src/bin/pathfinder/main.rs Outdated
byteZorvin and others added 2 commits November 14, 2025 23:03
…nd updating transaction module to use SettlementLayerAddress for from_address serialization.
Co-authored-by: Mohit Dhattarwal <48082542+Mohiiit@users.noreply.github.com>
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.

3 participants