search

vacp2p / rln-contract

RLN contract
Apache License 2.0
13 stars 6 forks source link

fix(rln): use zero-based indexing instead of 1-based indexing for commitments #28

Closed rymnc closed 1 year ago

rymnc commented 1 year ago

This messes with the tree merkle root, hence insertions should be from 0

github-actions[bot] commented 1 year ago

LCOV of commit 306a458 during CI #41

Summary coverage rate:
  lines......: 84.0% (499 of 594 lines)
  functions..: 50.0% (13 of 26 functions)
  branches...: 52.5% (21 of 40 branches)

Files changed coverage rate:
                              |Lines       |Functions  |Branches    
  Filename                    |Rate     Num|Rate    Num|Rate     Num
  ==================================================================
  contracts/RlnBase.sol       |97.2%     36|87.5%     8| 100%     18