Skip to content

Conversation

@samsonasik
Copy link
Member

@TomasVotruba when there are multi sprintf, and loop already return true early on VariableInSprintfMaskMatcher, this early mark as string, which should not, it should verify when there is sprintf with false mark in the loop.

@samsonasik
Copy link
Member Author

Fixed with count() check insted, only verify if there is single sprintf(), multi sprintf may cause need of collection of validations, which can be truthy early, then false later /cc @TomasVotruba

@samsonasik
Copy link
Member Author

/cc @staabm

@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it to have faster feedback to test ;)

@samsonasik samsonasik merged commit 1574a3b into main Oct 13, 2025
52 checks passed
@samsonasik samsonasik deleted the skip-multi-sprintf branch October 13, 2025 09:24
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.

2 participants