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

FastFail feature #1042 #1131

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Paxman23l
Copy link

Fixes Or Enhances

Adds failfast feature which returns the first validation error encountered. This can be set by a flag or function call. This PR was based on Issue 1042

Make sure that you've checked the boxes below before you submit PR:

  • Tests exist or have been written that cover this particular change.

@go-playground/validator-maintainers

@Paxman23l Paxman23l requested a review from a team as a code owner July 25, 2023 15:42
@coveralls
Copy link

coveralls commented Jul 25, 2023

Coverage Status

coverage: 75.295% (+0.02%) from 75.277%
when pulling b874bec on Paxman23l:paxman23l/fail-fast-feature
into 002f6c6 on go-playground:master.

@deankarn
Copy link
Contributor

deankarn commented Aug 6, 2023

interesting idea, let me think about the implications for a while :)

@Paxman23l Paxman23l force-pushed the paxman23l/fail-fast-feature branch from 02523cb to 57d69a8 Compare August 7, 2023 21:17
@xmgtony
Copy link

xmgtony commented Apr 15, 2024

Will this feature be merged? It's a very interesting and useful addition, and I'm looking forward to it.

@peczenyj
Copy link

peczenyj commented Dec 30, 2024

Hello

This feature will be wonderful in more than one project that I am working

In many cases I just need to fail fast. I will log error message and the content of the variable and I can figure out what happens later. And I don't need a deterministic error, can return any validation, does not matter.

do you need some help to test or merge this feature?

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.

5 participants