-
Notifications
You must be signed in to change notification settings - Fork 16
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.
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
[sbtc] Remove unused constants in sBTC token contract #1092
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR consists of removing unused constant, unused comments & minimal formatting changes (such as error codes in 'sbtc-token').
Only one question re: failed costs at 400 but looks good to me otherwise (assuming 'git is dirty' issue is fixed cc @djordon)
@friedger have you tried running |
I don't know why the check fails due to dirty git. Where get files changed? |
We run |
Description
Remove unused constants in sBTC token contract
Changes
Removes two constants from contract that were not used.
Removes check-checker and related comments to reduce size of contract
Testing Information
Unit test adapted.
More unit tests added.
Checklist: