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

Fix interpolation using %s instead of %#v for interfaces #1112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

effeix
Copy link

@effeix effeix commented Sep 2, 2021

Summary

Fix string interpolation issues using %s instead of %#v

Changes

Switch %s with %#v inside error messages in func Len()
use err.Error() instead of only err to log the error message in func NotEqual()
Switch %s with %#v inside error messages in func NotContains()
Switch %s with %#v inside error messages in func Subset()
Switch %s with %#v inside error messages in func NotSubset()

Motivation

The error messages were not clear on what was happening because the element is not being formatted correctly.

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.

None yet

2 participants