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

CVF-17 #184

Merged

kinrezC merged 2 commits into main from post-abdk-audit/17

Nov 3, 2024

Collaborator

clemlak commented Nov 2, 2024

Declaring top-level entities in a file named after a contract makes it harder navigating through code.

clemlak added 2 commits

November 2, 2024 15:25


          chore: move errors at the top level of the file

dd5ff20


          chore: remove unused error

466cae1

Collaborator Author

clemlak commented Nov 2, 2024

I found 2 unused errors but I left them because I want to actually use them.

kinrezC approved these changes

View reviewed changes

kinrezC merged commit 8dc5961 into main

1 check passed

kinrezC deleted the post-abdk-audit/17 branch

November 3, 2024 15:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet