Skip to content

Update FFI readme#1347

Merged
arminsabouri merged 1 commit intopayjoin:masterfrom
spacebear21:ffi-readme-updates
Feb 21, 2026
Merged

Update FFI readme#1347
arminsabouri merged 1 commit intopayjoin:masterfrom
spacebear21:ffi-readme-updates

Conversation

@spacebear21
Copy link
Collaborator

Include C# bindings, add links to published packages, and clean up some stale writing.

Include C# bindings, add links to published packages, and clean up some
stale writing.
@coveralls
Copy link
Collaborator

coveralls commented Feb 20, 2026

Pull Request Test Coverage Report for Build 22244027455

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.444%

Totals Coverage Status
Change from base Build 22242311720: 0.0%
Covered Lines: 10594
Relevant Lines: 12850

💛 - Coveralls

Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

Ack

This repository creates Payjoin libraries for various programming languages, all using the Rust-based [Payjoin Dev Kit](https://github.com/payjoin/rust-payjoin) as the core implementation of BIP-77.

Our mission is to provide developers with cross-language libraries that seamlessly integrate with different platform languages. By offering support for multiple languages, we aim to enhance the accessibility and usability of Payjoin, empowering developers to incorporate this privacy-enhancing feature into their applications, no matter their preferred programming language.
Our mission is to provide developers with cross-language libraries that seamlessly integrate with different platform languages. By offering support for multiple languages, we aim to enhance the accessibility and usability of Payjoin, empowering developers to incorporate it into their applications, no matter their preferred programming language.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this read better?

Suggested change
Our mission is to provide developers with cross-language libraries that seamlessly integrate with different platform languages. By offering support for multiple languages, we aim to enhance the accessibility and usability of Payjoin, empowering developers to incorporate it into their applications, no matter their preferred programming language.
Our mission is to provide developers with cross-language libraries that seamlessly integrate with different platform languages. By offering support for multiple languages, we aim to enhance the accessibility and usability of Payjoin, empowering developers to incorporate Payjoin into their applications, no matter their preferred programming language.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"...usability of Payjoin, empowering developers to incorporate Payjoin" I used "it" to avoid back to back repetition but it works either way imo

| Python | linux, macOS | [payjoin-ffi/python](python) | [pypi](https://pypi.org/project/payjoin) |
| Dart | linux, macOS | [payjoin-ffi/dart](dart) | [pub.dev](https://pub.dev/packages/payjoin) |
| JavaScript | linux, macOS | [payjoin-ffi/javascript](javascript) | [npm](https://www.npmjs.com/package/payjoin) |
| C# | linux, macOS, windows | [payjoin-ffi/csharp](csharp) | [nuget](https://www.nuget.org/packages/Payjoin) |
Copy link
Collaborator

Choose a reason for hiding this comment

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

OCD nit. Match with the other urls

Suggested change
| C# | linux, macOS, windows | [payjoin-ffi/csharp](csharp) | [nuget](https://www.nuget.org/packages/Payjoin) |
| C# | linux, macOS, windows | [payjoin-ffi/csharp](csharp) | [nuget](https://www.nuget.org/packages/payjoin) |

Copy link
Collaborator

Choose a reason for hiding this comment

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

That is how it is listed.. blame Dan for publishing it with a capital P lol

Copy link
Collaborator Author

@spacebear21 spacebear21 Feb 21, 2026

Choose a reason for hiding this comment

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

Blame C# for being weird, all their packages are capitalized https://www.nuget.org/packages

@arminsabouri
Copy link
Collaborator

The JS job is failing and it seems unrelated to this PR

error: failed to compile `wasm-bindgen-cli v0.2.110`, intermediate artifacts can be found at `/tmp/cargo-installr7gsOt`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

Caused by:
  rustc 1.85.1 is not supported by the following packages:
    time@0.3.47 requires rustc 1.88.0
    time-core@0.1.8 requires rustc 1.88.0

Is this a known issue?

@benalleng
Copy link
Collaborator

The JS job is failing and it seems unrelated to this PR

That's what #1349 addresses. We need to lock wasm-bindgen

@arminsabouri arminsabouri merged commit 2de3431 into payjoin:master Feb 21, 2026
42 of 48 checks passed
@spacebear21
Copy link
Collaborator Author

That's what #1349 addresses. We need to lock wasm-bindgen

Yep. Can I please get an ACK on that to unblock other failing PRs like #1350

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