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 syntax check failures #8028

Merged
merged 1 commit into from May 29, 2022
Merged

Conversation

weirdan
Copy link
Collaborator

@weirdan weirdan commented May 29, 2022

They were caused by installing packages with --ignore-platform-reqs
which brought PHP 8.1 autoloaded packages and caused fatal errors when
running php-parallel-lint. Instead of ignoring platform requirements we
now remove packages that are incompatible with PHP 7.1 (phpunit and its
dependents).

They were caused by installing packages with `--ignore-platform-reqs`
which brought PHP 8.1 autoloaded packages and caused fatal errors when
running php-parallel-lint. Instead of ignoring platform requirements we
now remove packages that are incompatible with PHP 7.1 (phpunit and its
dependents).
@weirdan weirdan mentioned this pull request May 29, 2022
@weirdan weirdan added the release:internal The PR will be included in 'Internal changes' section of the release notes label May 29, 2022
@weirdan
Copy link
Collaborator Author

weirdan commented May 29, 2022

Shepherd issue is fixed in #8029

@weirdan weirdan mentioned this pull request May 29, 2022
@weirdan weirdan requested a review from orklah May 29, 2022 04:49
@weirdan weirdan merged commit e47171d into vimeo:4.x May 29, 2022
@weirdan weirdan deleted the fix-syntax-check-failures branch May 29, 2022 19:57
@orklah
Copy link
Collaborator

orklah commented May 29, 2022

There's a build failure but I can't pinpoint what went wrong

@weirdan
Copy link
Collaborator Author

weirdan commented May 29, 2022

There's a build failure but I can't pinpoint what went wrong

I think that's the one that's fixed in #8029

@orklah orklah mentioned this pull request Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

2 participants