Reviewer Dashboard

@valentinewallace
3
PRs Assigned
5
Completed Reviews
12
Avg. Review Time (hours)

Pending Reviews

PR #3665 - lightning-invoice: explicitly enforce a 7089 B max length on BOLT11 invoice deser
Waiting for: 0 hours

Assigned PRs

PR #3655 - Expand `PaymentClaimable` to include all inbound channel IDs for a payment
PR #3665 - lightning-invoice: explicitly enforce a 7089 B max length on BOLT11 invoice deser
PR #3657 - Decrypt Trampoline error onions

Recent Completed Reviews

PR #3657 - Decrypt Trampoline error onions
Completed in: 8 hours
PR #3656 - Misc refactors and cleanups in advance of zero-fee commitments
Completed in: 19 hours
PR #3655 - Expand `PaymentClaimable` to include all inbound channel IDs for a payment
Completed in: 23 hours
PR #3645 - Add persistence test for htlc in the LocalRemoved state
Completed in: 2 hours
PR #3645 - Add persistence test for htlc in the LocalRemoved state
Completed in: 10 hours
@wpaulino
3
PRs Assigned
10
Completed Reviews
15
Avg. Review Time (hours)

Assigned PRs

PR #3651 - Add pending funding scopes to `FundedChannel`
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
PR #3641 - [Draft][Splicing] Add reserve check to splicing

Recent Completed Reviews

PR #3651 - Add pending funding scopes to `FundedChannel`
Completed in: 0 hours
PR #3641 - [Draft][Splicing] Add reserve check to splicing
Completed in: 8 hours
PR #3656 - Misc refactors and cleanups in advance of zero-fee commitments
Completed in: 14 hours
PR #3654 - Provide built commitment transactions to `ChannelMonitor`
Completed in: 10 hours
PR #3651 - Add pending funding scopes to `FundedChannel`
Completed in: 33 hours
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
Completed in: 32 hours
PR #3646 - Allow `ElectrumSyncClient` to reuse a preexisting client
Completed in: 0 hours
PR #3648 - Don't enforce owned values be written in `legacy` TLV writing
Completed in: 0 hours
PR #3643 - Trivial #3016 followups
Completed in: 1 hours
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
Completed in: 48 hours
@TheBlueMatt
7
PRs Assigned
15
Completed Reviews
9
Avg. Review Time (hours)

Pending Reviews

PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
Waiting for: 0 hours

Assigned PRs

PR #3639 - Introduce Flow utilities and OffersMessageFlow implementation
PR #3649 - Add BOLT12 support to bLIP-51 / LSPS1
PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
PR #3658 - Add idempotency check to prevent duplicate `InvoiceReceived` events
PR #3664 - Replace use of HolderSignedTx with HolderCommitment
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
PR #3640 - Require `FixedLengthReadable` for structs that always drain reader

Recent Completed Reviews

PR #3658 - Add idempotency check to prevent duplicate `InvoiceReceived` events
Completed in: 0 hours
PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
Completed in: 2 hours
PR #3664 - Replace use of HolderSignedTx with HolderCommitment
Completed in: 1 hours
PR #3654 - Provide built commitment transactions to `ChannelMonitor`
Completed in: 0 hours
PR #3659 - Pin once_cell@1.20.3 for older Rust versions
Completed in: 0 hours
PR #3650 - Attributable failures prefactor
Completed in: 1 hours
PR #3649 - Add BOLT12 support to bLIP-51 / LSPS1
Completed in: 19 hours
PR #3650 - Attributable failures prefactor
Completed in: 7 hours
PR #3650 - Attributable failures prefactor
Completed in: 8 hours
PR #3640 - Require `FixedLengthReadable` for structs that always drain reader
Completed in: 39 hours
@jkczyz
5
PRs Assigned
9
Completed Reviews
10
Avg. Review Time (hours)

Pending Reviews

PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
Waiting for: 0 hours

Assigned PRs

PR #3641 - [Draft][Splicing] Add reserve check to splicing
PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
PR #3658 - Add idempotency check to prevent duplicate `InvoiceReceived` events
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
PR #3639 - Introduce Flow utilities and OffersMessageFlow implementation

Recent Completed Reviews

PR #3641 - [Draft][Splicing] Add reserve check to splicing
Completed in: 5 hours
PR #3663 - Introduce FundingScope abstraction to ChannelMonitor
Completed in: 6 hours
PR #3658 - Add idempotency check to prevent duplicate `InvoiceReceived` events
Completed in: 20 hours
PR #3654 - Provide built commitment transactions to `ChannelMonitor`
Completed in: 2 hours
PR #3638 - Require counterparty_node_id TLV for ChannelMonitor
Completed in: 5 hours
PR #3638 - Require counterparty_node_id TLV for ChannelMonitor
Completed in: 9 hours
PR #3639 - Introduce Flow utilities and OffersMessageFlow implementation
Completed in: 0 hours
PR #3637 - DRAFT: Introduce interactive signing state flags for funded states.
Completed in: 22 hours
PR #3638 - Require counterparty_node_id TLV for ChannelMonitor
Completed in: 19 hours
@arik-so
2
PRs Assigned
11
Completed Reviews
13
Avg. Review Time (hours)

Assigned PRs

PR #3649 - Add BOLT12 support to bLIP-51 / LSPS1
PR #3640 - Require `FixedLengthReadable` for structs that always drain reader

Recent Completed Reviews

PR #3640 - Require `FixedLengthReadable` for structs that always drain reader
Completed in: 2 hours
PR #3661 - Use proper libm power method for `f64`s
Completed in: 1 hours
PR #3650 - Attributable failures prefactor
Completed in: 1 hours
PR #3650 - Attributable failures prefactor
Completed in: 8 hours
PR #3650 - Attributable failures prefactor
Completed in: 16 hours
PR #3650 - Attributable failures prefactor
Completed in: 24 hours
PR #3649 - Add BOLT12 support to bLIP-51 / LSPS1
Completed in: 40 hours
PR #3646 - Allow `ElectrumSyncClient` to reuse a preexisting client
Completed in: 0 hours
PR #3648 - Don't enforce owned values be written in `legacy` TLV writing
Completed in: 6 hours
PR #3640 - Require `FixedLengthReadable` for structs that always drain reader
Completed in: 48 hours
@joostjager
1
PRs Assigned
2
Completed Reviews
18
Avg. Review Time (hours)

Assigned PRs

PR #3657 - Decrypt Trampoline error onions

Recent Completed Reviews

PR #3657 - Decrypt Trampoline error onions
Completed in: 10 hours
PR #3657 - Decrypt Trampoline error onions
Completed in: 26 hours
@tnull
1
PRs Assigned
1
Completed Reviews
13
Avg. Review Time (hours)

Assigned PRs

PR #3662 - LSPS5 implementation

Recent Completed Reviews

PR #3662 - LSPS5 implementation
Completed in: 13 hours