-
Notifications
You must be signed in to change notification settings - Fork 680
Fix #8806 #8809
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
Conversation
Thanks! |
https://psalm.dev/r/fd479a295e - it still fails on psalm.dev |
I found these snippets: https://psalm.dev/r/fd479a295e<?php
array_map(null, [1]);
|
phar build seems broken, it doesn't launch. Maybe related to failed CI? |
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 |
Yeah but there's no artifact. Also, the CI on master branch says "Build phar / build-phar (push) Skipped" |
GH Actions build-phar.yml could be skipped if there are duplicate jobs running (because they would fail to push the results to |
Anyway it won't affect psalm.dev as we don't run phar there as far as I know. |
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...) |
@muglug can you check why |
Ah I've bumped this now |
No description provided.