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

chore: enable psl tests again #7596

Merged
merged 1 commit into from Feb 20, 2022
Merged

Conversation

azjezz
Copy link
Contributor

@azjezz azjezz commented Feb 6, 2022

released psalm-plugin 1.1.4, and 2.0.1 fixing compatibility with psalm 5.

I also rebased psalm/endtoend-test-psl repository on the latest 1.9.x changes ( which remove forbidEcho configuration entry )

@azjezz
Copy link
Contributor Author

azjezz commented Feb 6, 2022

test failure is correct, and this is the first psalm 5 bug discovered thanks to PSL :D

type iterable<Tsk:fn-psl\iter\iterator::create as mixed, Tsv:fn-psl\iter\iterator::create as mixed> does not contain Generator

@weirdan
Copy link
Collaborator

weirdan commented Feb 7, 2022

this is the first psalm 5 bug

Was there since 4.6.3, but is a bug anyway.

@orklah orklah added release:internal The PR will be included in 'Internal changes' section of the release notes PR: Need review labels Feb 7, 2022
@orklah
Copy link
Collaborator

orklah commented Feb 11, 2022

I think we can merge that and baseline the error while waiting for the fix. @weirdan do you mind?

Signed-off-by: azjezz <azjezz@protonmail.com>
@weirdan weirdan merged commit 6cc3da8 into vimeo:master Feb 20, 2022
@weirdan
Copy link
Collaborator

weirdan commented Feb 20, 2022

Thanks!

@azjezz azjezz deleted the chore/enable-psl branch February 20, 2022 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Need review release:internal The PR will be included in 'Internal changes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants