Skip to content

Use macros for Condition#760

Merged
reez merged 1 commit intobitcoindevkit:masterfrom
reez:condition
May 12, 2025
Merged

Use macros for Condition#760
reez merged 1 commit intobitcoindevkit:masterfrom
reez:condition

Conversation

@reez
Copy link
Copy Markdown
Collaborator

@reez reez commented May 12, 2025 •
edited
Loading

Description

Use macros for Condition

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

@reez reez changed the title refactor use macros for Condition Use macros for Condition May 12, 2025
@reez reez requested a review from thunderbiscuit May 12, 2025 15:52
thunderbiscuit
thunderbiscuit approved these changes May 12, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@thunderbiscuit thunderbiscuit left a comment

Choose a reason for hiding this comment

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

ACK e109c9c.

@reez reez merged commit 30ab852 into bitcoindevkit:master May 12, 2025
26 checks passed
@reez reez deleted the condition branch May 12, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thunderbiscuit thunderbiscuit thunderbiscuit approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@reez @thunderbiscuit