btcsuite / btcd

An alternative full node bitcoin implementation written in Go (golang)
https://github.com/btcsuite/btcd/blob/master/README.md
ISC License
6.2k stars 2.36k forks source link

fix: enhance code comments #2074

Closed threewebcode closed 9 months ago

threewebcode commented 9 months ago

Enhance and tidy the code comments

threewebcode commented 9 months ago

@davecgh, Please help review

coveralls commented 9 months ago

Pull Request Test Coverage Report for Build 7256586442

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report. To ensure accuracy in future PRs, please see these guidelines. A quick fix for this PR: rebase it; your next report should be accurate.


Files with Coverage Reduction New Missed Lines %
peer/peer.go 1 74.52%
netsync/blocklogger.go 9 0.0%
database/ffldb/db.go 25 91.67%
btcutil/gcs/builder/builder.go 28 68.04%
server.go 60 0.0%
blockchain/utxoviewpoint.go 93 73.76%
blockchain/chainio.go 170 61.25%
netsync/manager.go 187 0.0%
config.go 204 5.09%
blockchain/chain.go 222 73.38%
<!-- Total: 999 -->
Totals Coverage Status
Change from base Build 7227596643: 0.6%
Covered Lines: 28618
Relevant Lines: 50542

💛 - Coveralls
threewebcode commented 9 months ago

@davecgh, Please help review

threewebcode commented 9 months ago

@jcvernaleo The change set is updated with your review. Please take a look again.

jcvernaleo commented 9 months ago

@threewebcode I forgot to allow the github actions to run. Once those finish I'll merge.