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

feat: added require length check in propose #765

Merged

0xcuriousapple merged 1 commit into razor-network:master from GauravJain9:issue

Mar 25, 2022

Contributor

GauravJain9 commented Mar 25, 2022 •

edited

Loading

Fixes #759

Added ids.length == medians.length check in propose function
Negative test case written for that
Small fix in Internal Engine file


          feat: added require length check in propose

ec40d6f

coveralls commented Mar 25, 2022

Coverage increased (+0.005%) to 98.077% when pulling ec40d6f on GauravJain9:issue into 6469129 on razor-network:master.

0xcuriousapple approved these changes

View reviewed changes

SamAg19 approved these changes

View reviewed changes

0xcuriousapple merged commit 2535e59 into razor-network:master

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

Labels

None yet