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

test: add tests for associated type Record #7529

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

jaschdoc
Copy link
Member

@jaschdoc jaschdoc commented Apr 1, 2024

Supersedes mlutze#54

@magnus-madsen
Copy link
Member

@herluf-ba These tests reveal some issues in the new Weeder :)

@herluf-ba
Copy link
Contributor

Thanks 👍 I will have a look in the morning.

@herluf-ba
Copy link
Contributor

I made a pr that should fix the issue now 👍 I want to make you aware that the test is still failing due to ConstraintResolver. That's intended?

@magnus-madsen
Copy link
Member

I made a pr that should fix the issue now 👍 I want to make you aware that the test is still failing due to ConstraintResolver. That's intended?

Well, the tests are to find bugs in the ConstraintResolver, so I guess the answer is both yes and no :)

@jaschdoc
Copy link
Member Author

jaschdoc commented Apr 3, 2024

Still relevant: mlutze#54 (comment)

@mlutze
Copy link
Member

mlutze commented Apr 14, 2024

!!!rerun

Copy link

Rerunning 1 failed test(s)...

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

4 participants