Skip to content

feat: Add Nostr app - #70

Merged
akarve merged 5 commits into
akarve:mainfrom
ethicnology:main
Sep 13, 2026
Merged

akarve merged 5 commits into
akarve:mainfrom
ethicnology:main

Conversation

@ethicnology

Copy link
Copy Markdown
Contributor

@ethicnology

Copy link
Copy Markdown
Contributor Author

amended to make the PR working with poetry 1.X instead of 2.X

Comment thread src/bipsea/apps/nostr/app.py Outdated
Comment thread src/bipsea/apps/nostr/app.py Outdated
@akarve

akarve commented Jul 19, 2026

Copy link
Copy Markdown
Owner

@ethicnology howdy try the local linter as well. i approved CI and black is cranky:

Run make check
poetry run black . --check
would reformat /home/runner/work/bipsea/bipsea/src/bipsea/bipsea.py
would reformat /home/runner/work/bipsea/bipsea/tests/test_cli.py

Oh no! 💥 💔 💥
2 files would be reformatted, 31 files would be left unchanged.
make: *** [Makefile:49: check] Error 1
Error: Process completed with exit code 2.

Comment thread src/bipsea/apps/nostr/app.py
Drop the unused identity Param whose range disagreed with the Click option, and apply the project's Black formatting to the Nostr CLI and tests.
@akarve
akarve merged commit 2bb9d71 into akarve:main Sep 13, 2026
@akarve akarve mentioned this pull request Sep 13, 2026
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