Skip to content

NUT-21 and NUT-22 matching rules#334

Merged
thesimplekid merged 4 commits intocashubtc:mainfrom
robwoodgate:nut21-22-match
Feb 10, 2026
Merged

NUT-21 and NUT-22 matching rules#334
thesimplekid merged 4 commits intocashubtc:mainfrom
robwoodgate:nut21-22-match

Conversation

@robwoodgate
Copy link
Copy Markdown
Contributor

@robwoodgate robwoodgate commented Jan 27, 2026

Copy link
Copy Markdown
Collaborator

@thesimplekid thesimplekid left a comment

Choose a reason for hiding this comment

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

Looks good. Do you think some test cases would be helpful?

Comment thread 22.md Outdated
Comment thread 22.md Outdated
robwoodgate and others added 2 commits January 30, 2026 13:11
Co-authored-by: tsk <tsk@thesimplekid.com>
Co-authored-by: tsk <tsk@thesimplekid.com>
@robwoodgate
Copy link
Copy Markdown
Contributor Author

robwoodgate commented Jan 30, 2026

Looks good. Do you think some test cases would be helpful?

Thanks for removing the prefix anchor - I overlooked that.

Am happy to add tests, but wasn't sure a prefix match needs much more explanation than the example given?

But am too close to it, so if you think it's open to interpretation, then I will.

EDIT: 21a0bd3 adds a couple more examples

Copy link
Copy Markdown
Collaborator

@thesimplekid thesimplekid left a comment

Choose a reason for hiding this comment

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

LGTM. CDK PR cashubtc/cdk#1586

@robwoodgate
Copy link
Copy Markdown
Contributor Author

Cashu-TS PR: #334

@github-project-automation github-project-automation Bot moved this from Backlog to Done in nuts Feb 5, 2026
@robwoodgate robwoodgate reopened this Feb 5, 2026
@thesimplekid thesimplekid merged commit 014ed93 into cashubtc:main Feb 10, 2026
2 checks passed
@robwoodgate robwoodgate deleted the nut21-22-match branch February 10, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants