Skip to content

docs: validate#18

Merged
Naapperas merged 2 commits intoNaapperas:developfrom
cachho:docs/ReadmeGeneral
Mar 3, 2025
Merged

docs: validate#18
Naapperas merged 2 commits intoNaapperas:developfrom
cachho:docs/ReadmeGeneral

Conversation

@cachho
Copy link
Copy Markdown
Contributor

@cachho cachho commented Feb 5, 2025

validate isn't mentioned at all in the readme, you have to go to the examples to see it.

@Naapperas
Copy link
Copy Markdown
Owner

validate isn't mentioned at all in the readme, you have to go to the examples to see it.

Nice catch. I try to keep the README up to date but somethings always manage to get through.

While we are on this, it could be useful to add examples of all validation results, both for validate and for safe_validate. The PR is ready to be approved but please add those.

@cachho
Copy link
Copy Markdown
Contributor Author

cachho commented Feb 5, 2025

I'm curious, is there a partical reason you are calling it validator/validate instead of schema/parse?

@cachho cachho force-pushed the docs/ReadmeGeneral branch from 411239e to 7d47d20 Compare February 12, 2025 22:59
@cachho cachho force-pushed the docs/ReadmeGeneral branch from 7d47d20 to 3a504e1 Compare February 12, 2025 23:01
@Naapperas Naapperas merged commit 4cb0626 into Naapperas:develop Mar 3, 2025
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