Skip to content

Note that -w is now required for witnesses in simc#295

Merged
KyrylR merged 1 commit intoBlockstreamResearch:masterfrom
schoen:need-dash-w-option
Apr 23, 2026
Merged

Note that -w is now required for witnesses in simc#295
KyrylR merged 1 commit intoBlockstreamResearch:masterfrom
schoen:need-dash-w-option

Conversation

@schoen
Copy link
Copy Markdown
Contributor

@schoen schoen commented Apr 23, 2026

I believe this changed in SimplicityHL 0.5.0.

@schoen schoen requested a review from delta1 as a code owner April 23, 2026 14:28
Copy link
Copy Markdown
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

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

ACK 27c28c6

UPD: ci failing

@schoen
Copy link
Copy Markdown
Contributor Author

schoen commented Apr 23, 2026

I thought I specified -S but apparently I specified -s instead. Oops!

@schoen schoen force-pushed the need-dash-w-option branch from 27c28c6 to 2855e9a Compare April 23, 2026 14:58
Copy link
Copy Markdown
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

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

ACK 2855e9a

@KyrylR KyrylR merged commit 8a370fa into BlockstreamResearch:master Apr 23, 2026
11 checks passed
@schoen schoen deleted the need-dash-w-option branch April 23, 2026 16:38
Copy link
Copy Markdown
Contributor

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 2855e9a; successfully ran local tests

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.

3 participants