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

add smaug plugin #644

Merged
merged 2 commits into from
Feb 5, 2025
Merged

add smaug plugin #644

merged 2 commits into from
Feb 5, 2025

Conversation

chrisguida
Copy link
Collaborator

This plugin sends bkpr-compatible events to the bkpr plugin for external on-chain wallet movements. This allows bkpr to maintain a full picture of a merchant's bitcoin inflows and outflows, vastly improving bitcoin accounting and financial analysis.

@chrisguida chrisguida self-assigned this Jan 28, 2025
@chrisguida chrisguida force-pushed the smaug branch 9 times, most recently from 8e3d8a5 to 110f380 Compare January 31, 2025 20:14
This is to avoid the case where a Rust plugin has a pip
requirements file or a pyproject.toml file for testing purposes,
but we still want to run the generic tests/setup.sh script
rather than attempting to build a Python plugin.
@chrisguida
Copy link
Collaborator Author

Okay, the CI is passing! I needed to update the CI code in order to handle the case where a Rust plugin has a Poetry project for testing purposes... obviously we still want to run the generic setup.sh in that case!

Another note: It's expected that smaug won't work on CLN 24.05 and before, since the PR allowing smaug to work was not released until 24.08.

@daywalker90
Copy link
Contributor

ACK 1200f60

Copy link
Contributor

@cdecker cdecker left a comment

Choose a reason for hiding this comment

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

🚀

@chrisguida
Copy link
Collaborator Author

Hey @cdecker @vincenzopalazzo could one of you merge this please? Would rather not self-merge my own plugin :)

@vincenzopalazzo vincenzopalazzo merged commit 8a3eef7 into lightningd:master Feb 5, 2025
5 of 7 checks passed
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