Skip to content

ffi: Extract shared InMemory persister helpers#1553

Merged
spacebear21 merged 1 commit into
payjoin:masterfrom
spacebear21:ffi-extract-shared-in-memory-persisters
May 11, 2026
Merged

ffi: Extract shared InMemory persister helpers#1553
spacebear21 merged 1 commit into
payjoin:masterfrom
spacebear21:ffi-extract-shared-in-memory-persisters

Conversation

@spacebear21

Copy link
Copy Markdown
Collaborator

Addresses #1529 somewhat (the suggested approach is misguided, per #1533 (comment))

Supersedes #1533

Authored with Codex 5.5

Pull Request Checklist

Please confirm the following before requesting review:

@coveralls

coveralls commented May 11, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 25698285159

Coverage remained the same at 85.294%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 13668
Covered Lines: 11658
Line Coverage: 85.29%
Coverage Strength: 395.84 hits per line

💛 - Coveralls

@spacebear21

Copy link
Copy Markdown
Collaborator Author

cc @ValeraFinebits

@ValeraFinebits

Copy link
Copy Markdown
Contributor

cc @ValeraFinebits

Looks good to me. If build and tests pass, I'm good with it.

@spacebear21

Copy link
Copy Markdown
Collaborator Author

Tests are failing due to an unrelated dependency issue, I have a fix in #1541

@spacebear21 spacebear21 force-pushed the ffi-extract-shared-in-memory-persisters branch 2 times, most recently from f6028a9 to ec8f221 Compare May 11, 2026 21:14
@spacebear21 spacebear21 marked this pull request as ready for review May 11, 2026 21:17
@spacebear21 spacebear21 requested review from benalleng and chavic May 11, 2026 21:17
@spacebear21 spacebear21 force-pushed the ffi-extract-shared-in-memory-persisters branch from ec8f221 to ecd8a8f Compare May 11, 2026 21:26

@benalleng benalleng left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK ecd8a8f

@spacebear21 spacebear21 merged commit 88b21aa into payjoin:master May 11, 2026
23 checks passed
@benalleng benalleng mentioned this pull request Jun 7, 2026
17 tasks
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.

4 participants