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

Fixed generic assertions for list and array #7076

Merged
merged 1 commit into from Dec 7, 2021
Merged

Fixed generic assertions for list and array #7076

merged 1 commit into from Dec 7, 2021

Conversation

klimick
Copy link
Contributor

@klimick klimick commented Dec 5, 2021

fix #5294

@weirdan weirdan added the release:fix The PR will be included in 'Fixes' section of the release notes label Dec 5, 2021
@weirdan weirdan changed the title Try to fix generic assertions for list and array Fixed generic assertions for list and array Dec 5, 2021
@weirdan weirdan requested a review from orklah December 5, 2021 23:58
@orklah
Copy link
Collaborator

orklah commented Dec 6, 2021

I'm a little out of my depth here and I'm not sure what side effects could appear. However, I tested that each change is indeed needed to fix every test case added, the changes are quite small and seems coherent so I'd go for it...

@weirdan weirdan merged commit 2530b33 into vimeo:master Dec 7, 2021
@weirdan
Copy link
Collaborator

weirdan commented Dec 7, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User defined assertions works incorrectly for generic iterable types
3 participants