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

Fix #8806 #8809

Merged
merged 3 commits into from Dec 1, 2022
Merged

Fix #8806 #8809

merged 3 commits into from Dec 1, 2022

Conversation

danog
Copy link
Collaborator

@danog danog commented Dec 1, 2022

No description provided.

@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Dec 1, 2022
@orklah orklah merged commit 433e272 into vimeo:master Dec 1, 2022
@orklah
Copy link
Collaborator

orklah commented Dec 1, 2022

Thanks!

@weirdan
Copy link
Collaborator

weirdan commented Dec 1, 2022

https://psalm.dev/r/fd479a295e - it still fails on psalm.dev

@psalm-github-bot
Copy link

I found these snippets:

https://psalm.dev/r/fd479a295e
<?php

array_map(null, [1]);
Psalm encountered an internal error:

/vendor/vimeo/psalm/src/Psalm/Internal/Provider/ReturnTypeProvider/ArrayMapReturnTypeProvider.php: Psalm\Internal\Provider\ReturnTypeProvider\ArrayMapReturnTypeProvider::Psalm\Internal\Provider\ReturnTypeProvider\{closure}(): Argument #1 ($sub) must be of type array, Psalm\Type\Union given

@orklah
Copy link
Collaborator

orklah commented Dec 1, 2022

phar build seems broken, it doesn't launch. Maybe related to failed CI?

@weirdan
Copy link
Collaborator

weirdan commented Dec 1, 2022

it doesn't launch

Doesn't it? I see a non-crashing run right here in this ticket: https://app.circleci.com/pipelines/github/vimeo/psalm/10128/workflows/d1c6c5c6-09e3-49ca-bd07-e0656013a83d/jobs/35716

@orklah
Copy link
Collaborator

orklah commented Dec 1, 2022

Yeah but there's no artifact. Also, the CI on master branch says "Build phar / build-phar (push) Skipped"

@weirdan
Copy link
Collaborator

weirdan commented Dec 1, 2022

psalm.phar artifact is produced in a previous job (of the same workflow): https://app.circleci.com/pipelines/github/vimeo/psalm/10128/workflows/d1c6c5c6-09e3-49ca-bd07-e0656013a83d/jobs/35715/artifacts

GH Actions build-phar.yml could be skipped if there are duplicate jobs running (because they would fail to push the results to psalm/phar): https://github.com/vimeo/psalm/blob/master/.github/workflows/build-phar.yml#L15-L29

@weirdan
Copy link
Collaborator

weirdan commented Dec 1, 2022

Anyway it won't affect psalm.dev as we don't run phar there as far as I know.

@orklah
Copy link
Collaborator

orklah commented Dec 2, 2022

I thought it did for running psalm.

Anyway, Psalm.dev is running with commit c3cc906 which is outdated (but I'm not sure why now...)

@weirdan
Copy link
Collaborator

weirdan commented Dec 2, 2022

@muglug can you check why vimeo/psalm does not get updated on psalm.dev? I don't think anyone else can do that.

@muglug
Copy link
Collaborator

muglug commented Dec 2, 2022

Ah I've bumped this now

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.

None yet

4 participants