Skip to content

nostr: rework SingleLetterTag - #1416

Merged
yukibtc merged 1 commit into
masterfrom
single-letter-tag
Aug 3, 2026
Merged

nostr: rework SingleLetterTag#1416
yukibtc merged 1 commit into
masterfrom
single-letter-tag

Conversation

@yukibtc

@yukibtc yukibtc commented Aug 3, 2026

Copy link
Copy Markdown
Member

Checklist

  • I followed the contribution guidelines
  • I updated the relevant CHANGELOG.md (if applicable)
  • I understand and can explain all code in this PR

@yukibtc yukibtc added this to the v0.45 milestone Aug 3, 2026
@yukibtc
yukibtc requested a review from TheAwiteb as a code owner August 3, 2026 09:33
yukibtc added a commit that referenced this pull request Aug 3, 2026
Pull-Request: #1416
Signed-off-by: Yuki Kishimoto <yukikishimoto@protonmail.com>
@yukibtc
yukibtc force-pushed the single-letter-tag branch from bdcda04 to bb9376a Compare August 3, 2026 09:33
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.97561% with 39 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
nostr/src/filter/single_letter.rs 81.98% 29 Missing ⚠️
nostr/src/filter/mod.rs 74.07% 7 Missing ⚠️
nostr-sdk/src/client/gossip/mod.rs 0.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread nostr/src/filter/single_letter.rs
Comment thread nostr/src/filter/single_letter.rs Outdated
Comment thread nostr/src/filter/single_letter.rs Outdated
Pull-Request: #1416
Signed-off-by: Yuki Kishimoto <yukikishimoto@protonmail.com>
@yukibtc
yukibtc force-pushed the single-letter-tag branch from bb9376a to e17d7d8 Compare August 3, 2026 09:46

@TheAwiteb TheAwiteb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK

@yukibtc
yukibtc merged commit e17d7d8 into master Aug 3, 2026
42 checks passed
@yukibtc
yukibtc deleted the single-letter-tag branch August 3, 2026 09:50
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