Skip to content

Add method Skip.When() #123

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 13, 2020
Merged

Add method Skip.When() #123

merged 1 commit into from
Aug 13, 2020
+41 −27

Conversation

@maratori
Copy link
Contributor

@maratori maratori commented Aug 10, 2020

Fix #122

Add method Skip.When() to be able skip validation based on condition.
It' necessary in cases of nested structs validation. Because When(..., Skip) is not the same as Skip.When(...) (see #122).
@maratori
Add method Skip.When()
06f4b6f 
Skip.When() determines if all rules following it should be skipped
@coveralls
Copy link

coveralls commented Aug 10, 2020

Coverage Status

Coverage increased (+0.003%) to 99.336% when pulling 06f4b6f on maratori:skip-when into 6a5d431 on go-ozzo:master.
@qiangxue
Copy link
Member

qiangxue commented Aug 13, 2020

Excellent work! Thank you very much!
@qiangxue qiangxue merged commit b589ba5 into go-ozzo:master Aug 13, 2020
@maratori maratori deleted the skip-when branch August 15, 2020 22:01
ltns35 pushed a commit to ltns35/ozzo-validation that referenced this pull request Jun 5, 2022
@maratori
Add method Skip.When() (go-ozzo#123)
dc3fb21 
Skip.When() determines if all rules following it should be skipped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@maratori @coveralls @qiangxue