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

feat: add exception for mime.ErrInvalidMediaParameter #79

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

kolyshkin
Copy link
Contributor

It is documented (1) that mime.ParseMediaType can return mime.ErrInvalidMediaParameter error, so it's not a bug to perform a direct comparison.

It is documented ([1]) that mime.ParseMediaType can return
mime.ErrInvalidMediaParameter error, so it's not a bug to perform a
direct comparison.

[1]: https://pkg.go.dev/mime#ParseMediaType
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin
Copy link
Contributor Author

CI / conventional-commits failure is not an issue with this commit, but with #77 (not sure why it was not detected in that PR).

Also, a CI check should only check the given commit (I'll open a PR).

@kolyshkin
Copy link
Contributor Author

Also, a CI check should only check the given commit (I'll open a PR).

See #80

@polyfloyd polyfloyd merged commit 24f58bd into polyfloyd:main Aug 6, 2024
1 of 2 checks passed
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