Skip to content

Change assert to ret false (verify)#36

Merged
tcoratger merged 3 commits intomainfrom
asserts-to-ret-false
Jan 22, 2026
Merged

Change assert to ret false (verify)#36
tcoratger merged 3 commits intomainfrom
asserts-to-ret-false

Conversation

@b-wagn
Copy link
Contributor

@b-wagn b-wagn commented Jan 22, 2026

Addresses #34 and #35. Thanks for spotting @this-vishalsingh

Copy link
Contributor

@tcoratger tcoratger left a comment

Choose a reason for hiding this comment

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

LGTM, just tiny typo.

Co-authored-by: Thomas Coratger <60488569+tcoratger@users.noreply.github.com>
@tcoratger tcoratger merged commit 380301e into main Jan 22, 2026
2 checks passed
@this-vishalsingh
Copy link

Hey @tcoratger, thanks looks good!

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