Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jade pip api and native psbt #763

Conversation

JamieDriver
Copy link
Contributor

As of 1.0.32, the jade-client (jadepy) is available on PyPi and so can be pip install-ed, and we can remove the local copy of the code.

If preferred, this can supersede #753

@JamieDriver JamieDriver force-pushed the jade_pip_api_and_native_psbt branch from f8d5edb to 2d96b1f Compare October 14, 2024 09:40
@JamieDriver JamieDriver changed the title WIP: Jade pip api and native psbt Jade pip api and native psbt Oct 23, 2024
@JamieDriver JamieDriver mentioned this pull request Oct 23, 2024
@JamieDriver JamieDriver force-pushed the jade_pip_api_and_native_psbt branch from 2d96b1f to 2fab146 Compare November 7, 2024 08:58
@JamieDriver
Copy link
Contributor Author

Hi,
Are you waiting for anything further from me before you can consider this PR ?
I've rebased it onto current master (3.1.0).
Thanks,
J.

@JamieDriver JamieDriver force-pushed the jade_pip_api_and_native_psbt branch from 2fab146 to 28c4e8d Compare November 7, 2024 12:13
@achow101
Copy link
Member

achow101 commented Dec 4, 2024

NACK

We prefer to pull in vendor libraries to have better control over dependencies and to be able to remove things that we do not wish to support. In this particular case, that would be the bluetooth stuff.

@achow101 achow101 closed this Dec 4, 2024
@JamieDriver
Copy link
Contributor Author

The pip install does not include the 'Bluetooth stuff' !

But ok, if you prefer to include a copy of the lib code, I'll revisit #753.

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