Skip to content

Slog1#166

Closed
ellemouton wants to merge 10 commits intomasterfrom
slog1
Closed

Slog1#166
ellemouton wants to merge 10 commits intomasterfrom
slog1

Conversation

@ellemouton
Copy link
Copy Markdown
Owner

Change Description

Description of change / link to associated issue.

Steps to Test

Steps for reviewers to follow to test the change.

Pull Request Checklist

Testing

  • Your PR passes all CI checks.
  • Tests covering the positive and negative (error paths) are included.
  • Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

📝 Please see our Contribution Guidelines for further guidance.

And instead use the new btclog Logger `WithPrefix` method.
@github-actions
Copy link
Copy Markdown

Pull reviewers stats

Stats of the last 30 days for lnd:

User Total reviews Time to review Total comments

@coveralls
Copy link
Copy Markdown

coveralls commented Nov 27, 2024

Pull Request Test Coverage Report for Build 12046326611

Details

  • 51 of 129 (39.53%) changed or added relevant lines in 11 files are covered.
  • 280 unchanged lines in 35 files lost coverage.
  • Overall coverage decreased (-0.5%) to 58.462%

Changes Missing Coverage Covered Lines Changed/Added Lines %
build/version.go 5 10 50.0%
lnd.go 23 96 23.96%
Files with Coverage Reduction New Missed Lines %
htlcswitch/linkfailure.go 2 58.62%
queue/gc_queue.go 2 96.51%
watchtower/wtclient/session_negotiator.go 2 79.68%
lnwallet/sigpool.go 2 67.71%
htlcswitch/switch.go 2 83.42%
sweep/sweeper.go 2 89.96%
chainntnfs/txnotifier.go 2 88.59%
input/signdescriptor.go 2 63.49%
routing/payment_session.go 2 81.89%
watchtower/wtclient/queue.go 3 87.0%
Totals Coverage Status
Change from base Build 12043192977: -0.5%
Covered Lines: 93250
Relevant Lines: 159506

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants