transparency-dev / transparency-dev/merkle
Repo spring sweep
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Go
- Stars
- 41
- Forks
- 13
- Avg merge
- 19h 29m
- Merged PRs (30d)
- 4
Description
Working on #225 revealed that the repo could do with some cleanups:
- remove go1.18 directives in
tree_fuzz_testandnode_fuzz_test.go - bump the go version up
- add toolchain directive
- more idiomatic tests (see this PR comment)
- use range for loops were possible
- fix codecov which uses a very old base and keeps reporting error on CLs
- make reference inplementation of consistency and inclusion proof test signatures match RFCs
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the Go version directives in tree_fuzz_test and node_fuzz_test.go, then review the linked PR comment and locate the Codecov configuration and reference proof implementation. Done means each unchecked checklist item is addressed, including matching the reference test signatures to the RFCs and resolving the Codecov reporting error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- build-system, devtools, testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100