Skip to content

Make 0.14 CI green again #919

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

Merged
merged 10 commits into from
Nov 23, 2021
Merged

Make 0.14 CI green again #919

merged 10 commits into from
Nov 23, 2021

Conversation

ruudk
Copy link
Contributor

@ruudk ruudk commented Nov 23, 2021

Best to review per commit.

@ruudk ruudk changed the title Enable validator in tests Fix coverage job Nov 23, 2021
@ruudk ruudk force-pushed the fix-coverage branch 7 times, most recently from c2eaad0 to 728da1d Compare November 23, 2021 14:09
Conditions should be quoted.
This requires `doctrine/annotations` to be installed.
Shouldn't be needed.
It's the default.
Much easier to use as it caches automatically.

And it makes it possible to test lower and highest dependencies in the future.
Always use strings and no need for asterisk.
@ruudk ruudk force-pushed the fix-coverage branch 3 times, most recently from aeef68c to c868a21 Compare November 23, 2021 14:15
@ruudk ruudk changed the title Fix coverage job Make CI green again Nov 23, 2021
@ruudk ruudk requested review from mcg-web, Vincz and murtukov November 23, 2021 14:17
@mcg-web mcg-web merged commit 7ff212e into overblog:0.14 Nov 23, 2021
@ruudk ruudk changed the title Make CI green again Make 0.14 CI green again Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants