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

More attribute fixes. #7788

Merged
merged 2 commits into from
Mar 14, 2022
Merged

Conversation

AndrolGenhald
Copy link
Collaborator

AttributeStorage isn't really used anymore, but it's still around for plugins.

@AndrolGenhald AndrolGenhald added the release:fix The PR will be included in 'Fixes' section of the release notes label Mar 14, 2022
@AndrolGenhald
Copy link
Collaborator Author

AndrolGenhald commented Mar 14, 2022

Any idea what's up with the phar build? It doesn't seem related to this PR, but other recent PRs haven't had it fail.

Also, let me know when this is merged to master and I'll do another PR for class constant attributes. I figure it's easier to wait for Psalm 5 to do that since class const analysis changed significantly.

@orklah
Copy link
Collaborator

orklah commented Mar 14, 2022

It has been broken for a few days. Symfony added a polyfill for PhpToken I believe and the Phar build doesn't like that. I haven't looked how to fix that for now

@orklah orklah merged commit 32f10c3 into vimeo:4.x Mar 14, 2022
@orklah
Copy link
Collaborator

orklah commented Mar 14, 2022

Thanks!

@orklah
Copy link
Collaborator

orklah commented Mar 14, 2022

Oh, I think the phar build has been fixed by Matt in fact

@AndrolGenhald
Copy link
Collaborator Author

Oh, yeah, I see it, I didn't rebase on master before pushing so this PR is actually "older" than the other recent PRs, that explains it.

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
2 participants