Skip to content

#34 Support regex validation of property values #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Aug 8, 2022
Merged

Conversation

vincent-zurczak
Copy link
Member

Extend the syntax to verify property values against a regular expression.
BTW, I may ask one of my colleagues to perform reviews on this project, if no one else feels comfortable on the library and its usage. In which case we should discuss how this person could join the group.

@vincent-zurczak vincent-zurczak requested a review from a team May 28, 2022 19:24
@vincent-zurczak
Copy link
Member Author

@pascallibenzi Changes done.

@vincent-zurczak
Copy link
Member Author

Hello @martin-schulze-vireso.
Reviewed and ready to be merged. Thanks!

@vincent-zurczak vincent-zurczak force-pushed the #34 branch 4 times, most recently from 857340e to ecdad4b Compare July 20, 2022 16:07
@martin-schulze-vireso
Copy link
Member

martin-schulze-vireso commented Jul 20, 2022

Hm, it seems I can only force thenmerge but the bats-detik team must approve for you to merge. I am currently on my phone so I cannot dig deeper. I will investigate this later.

@vincent-zurczak
Copy link
Member Author

In fact, my last commit was not signed.
I am trying to fix it.

@vincent-zurczak
Copy link
Member Author

Ready to be merged. @martin-schulze-vireso
Thanks!

@vincent-zurczak vincent-zurczak merged commit 0d71702 into master Aug 8, 2022
@vincent-zurczak vincent-zurczak deleted the #34 branch March 27, 2023 07:07
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