Skip to content

Conversation

fatelei
Copy link
Contributor

@fatelei fatelei commented Dec 17, 2024

Fixes Or Enhances

fix issue #763

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

@fatelei fatelei requested a review from a team as a code owner December 17, 2024 14:40
@coveralls
Copy link

coveralls commented Dec 17, 2024

Coverage Status

coverage: 74.318%. remained the same
when pulling d7356f1 on fatelei:issue-743
into 6c3307e on go-playground:master.

@nodivbyzero
Copy link
Contributor

How does this change fix the issue #763?
image

Copy link
Contributor

@deankarn deankarn left a comment

Choose a reason for hiding this comment

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

LGTM now that it's using errors.As

@nodivbyzero nodivbyzero merged commit e564451 into go-playground:master Feb 12, 2025
15 checks passed
deankarn pushed a commit that referenced this pull request Feb 18, 2025
…be interface or implement error" in examples. (#1378)

Only *InvalidValidationError implements error, so we must use double
pointer.
good explanation:
https://www.reddit.com/r/golang/comments/txi397/comment/i3lybab/?utm_source=share&utm_medium=web3x&utm_name=web3xcss&utm_term=1&utm_content=share_button
and https://stackoverflow.com/a/69448087/9066110

## Fixes Or Enhances
Fixes `panic: errors: *target must be interface or implement error` for
`_examples/simple/` and `_examples/struct-level/` that was introduced in
#1346

**Make sure that you've checked the boxes below before you submit PR:**
- [x] Tests exist or have been written that cover this particular
change.

@go-playground/validator-maintainers
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.

6 participants