Skip to content

Conversation

@LukeMathWalker
Copy link
Contributor

Prevent "ambiguous associated type" if one of the variants is named Numeric.

@meithecatte
Copy link
Owner

This looks good. Sorry for the oversight 😅

I've rebased the PR onto master to fix a CI failure, which was caused by a recent nightly changing the wording of the error messages, which invalidated the ui tests.

With regards to the bundled formatting changes – if you insist on them, could you put them in a separate commit?

Prevent "ambiguous associated type" if one of the variants is named `Numeric`.
@LukeMathWalker
Copy link
Contributor Author

My bad, that's the autoformatter kicking in 😅

@meithecatte meithecatte merged commit 035c027 into meithecatte:master Jun 8, 2025
4 checks passed
@LukeMathWalker LukeMathWalker deleted the patch-1 branch June 8, 2025 15:09
@chesedo
Copy link

chesedo commented Jun 9, 2025

@meithecatte is it possible to release a new version with this fix?

@meithecatte
Copy link
Owner

@meithecatte is it possible to release a new version with this fix?

Just to let you know, I've done a release yesterday. I'm just running with very little time to be doing things.

@chesedo
Copy link

chesedo commented Jun 11, 2025

Ah sorry, I was only checking the GitHub release page 🙈 . Thanks Maja!

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.

3 participants