Skip to content

Conversation

@MohammadNassar1
Copy link
Collaborator

@MohammadNassar1 MohammadNassar1 commented Sep 21, 2025

This change is Reviewable

Copy link
Collaborator

@ishay-starkware ishay-starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

@ishay-starkware reviewed all commit messages.
Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @MohammadNassar1)

@MohammadNassar1 MohammadNassar1 changed the base branch from mohammad/audit-issues/add-fee-to-events to main September 25, 2025 09:06
@MohammadNassar1 MohammadNassar1 dismissed ishay-starkware’s stale review September 25, 2025 09:06

The base branch was changed.

@MohammadNassar1 MohammadNassar1 force-pushed the mohammad/audit-issues/check-zero-amount-before-transfer branch from d56a8ff to f9d8b8e Compare September 25, 2025 09:10
Copy link
Collaborator

@ishay-starkware ishay-starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

@ishay-starkware reviewed 1 of 1 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @MohammadNassar1)

@MohammadNassar1 MohammadNassar1 merged commit d8bbe8e into main Sep 25, 2025
3 checks passed
@MohammadNassar1 MohammadNassar1 deleted the mohammad/audit-issues/check-zero-amount-before-transfer branch September 25, 2025 09:13
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