Skip to content

fix: strictVerify not working with true value#345

Merged
BlackHole1 merged 2 commits intoelectron:mainfrom
t3chguy:t3chguy/fix/344
Mar 5, 2025
Merged

fix: strictVerify not working with true value#345
BlackHole1 merged 2 commits intoelectron:mainfrom
t3chguy:t3chguy/fix/344

Conversation

@t3chguy
Copy link
Copy Markdown
Contributor

@t3chguy t3chguy commented Feb 3, 2025

And extend it to support string-specified restrictions

Fixes #344

And extend it to support string-specified restrictions
@t3chguy t3chguy marked this pull request as ready for review February 3, 2025 21:34
@t3chguy t3chguy requested a review from a team as a code owner February 3, 2025 21:34
@erickzhao erickzhao self-requested a review February 3, 2025 22:11
Copy link
Copy Markdown
Member

@BlackHole1 BlackHole1 left a comment

Choose a reason for hiding this comment

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

LGTM

@t3chguy
Copy link
Copy Markdown
Contributor Author

t3chguy commented Feb 25, 2025

What is left to do here?

@BlackHole1
Copy link
Copy Markdown
Member

@t3chguy Can you fix test unit? 🙇‍♂️

@BlackHole1
Copy link
Copy Markdown
Member

Hey @t3chguy 👋. I noticed that @erickzhao has set himself as a reviewer, so I think we should wait for him to do the code review as well. However, he is currently on vacation, so we may have to wait until he returns next week.

@t3chguy
Copy link
Copy Markdown
Contributor Author

t3chguy commented Feb 28, 2025

No rush from my end

@BlackHole1 BlackHole1 merged commit 087ddb6 into electron:main Mar 5, 2025
@continuous-auth
Copy link
Copy Markdown

🎉 This PR is included in version 1.3.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

strictVerify types seem wrong

3 participants