Skip to content

fix(svm): N-02 remove unused errors #841

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 20, 2025
Merged

fix(svm): N-02 remove unused errors #841

merged 1 commit into from
Jan 20, 2025

Conversation

@Reinis-FRP
Copy link
Contributor

@Reinis-FRP Reinis-FRP commented Jan 7, 2025

OZ identified following issue:

Throughout the codebase, multiple instances of unused errors were identified:

- InvalidRoutePDA
- NotFilled
- InvalidUint128

Consider removing any unused errors.

This PR addresses the issue by removing unused errors.

@Reinis-FRP
fix(svm): N-02 remove unused errors
6ac2394 
Signed-off-by: Reinis Martinsons <reinis@umaproject.org>
@linear
Copy link

linear bot commented Jan 7, 2025

ACX-3591 N-02 Unused Errors

@Reinis-FRP Reinis-FRP merged commit a242dec into master Jan 20, 2025
9 checks passed
@Reinis-FRP Reinis-FRP deleted the reinis/acx-3591-n-02-unused-errors branch January 20, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrismaree chrismaree chrismaree approved these changes

@md0x md0x md0x approved these changes

@mrice32 mrice32 Awaiting requested review from mrice32 mrice32 is a code owner

@nicholaspai nicholaspai Awaiting requested review from nicholaspai nicholaspai is a code owner

Assignees

No one assigned

Labels

OZ Audit - SVM December 2024

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Reinis-FRP @chrismaree @md0x