Skip to content
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

feat: uuid v4 validation matcher #1248

Merged

Conversation

phos-web
Copy link
Contributor

added validation matcher which checks if value is a UUID v4

Copy link
Collaborator

@bbortt bbortt left a comment

Choose a reason for hiding this comment

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

thank you for your contribution. I've only left two small remarks.

@bbortt bbortt added Type: Feature Prio: Medium good first issue A good point to start from for new contributrs State: Review If pull-request has been opened an is ready/in review labels Nov 2, 2024
@phos-web phos-web force-pushed the feat/TAT-1196-uuid-validation branch from dde5ee1 to a9f25fc Compare November 6, 2024 09:22
@phos-web phos-web force-pushed the feat/TAT-1196-uuid-validation branch from a9f25fc to 48488de Compare November 6, 2024 13:10
Copy link
Collaborator

@bbortt bbortt left a comment

Choose a reason for hiding this comment

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

/approve

@bbortt bbortt merged commit ca50d7e into citrusframework:main Nov 7, 2024
1 check passed
bbortt added a commit to postfinance/citrus that referenced this pull request Nov 10, 2024
bbortt added a commit that referenced this pull request Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue A good point to start from for new contributrs Prio: Medium State: Review If pull-request has been opened an is ready/in review Type: Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants