Skip to content

Fixing tests for Symfony5 #499

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 1 commit into from
Dec 5, 2019
Merged

Conversation

weaverryan
Copy link
Member

This also removes the dev-stable. That was a build to detect possible BC breaks on patch versions made in Symfony... which is rare and probably not the job of Maker :).

@weaverryan weaverryan force-pushed the matrix-sf4-sf5 branch 3 times, most recently from 975dcf9 to 121ae77 Compare November 25, 2019 22:07
@weaverryan weaverryan changed the title tweaking the test matrix to test Symfony 4, Symfony 5 & change DEV to allow_failure Fixing tests for Symfony5 Nov 25, 2019
* tweaking the test matrix to test Symfony 4, Symfony 5 & change DEV to allow_failure
* Fixing arrangement of createClient() and booting the kernel
* Fix bug where object type-hint in listener tricked make:subscriber
* making test work for PHP 7.1 and lower
* avoiding 3 tests in PHP 7.1 due to seg-faulting
* fixing phpcs
* adding a workaround for an odd failing test
* fixing the dev version tests
weaverryan added a commit that referenced this pull request Dec 5, 2019
This PR was merged into the 1.0-dev branch.

Discussion
----------

Fixing tests for Symfony5

This also removes the dev-stable. That was a build to detect possible BC breaks on patch versions made in Symfony... which is rare and probably not the job of Maker :).

Commits
-------

56df019 Various test fixes & matrix improvements:
@weaverryan weaverryan merged commit 56df019 into symfony:master Dec 5, 2019
@weaverryan weaverryan deleted the matrix-sf4-sf5 branch December 5, 2019 14:27
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.

1 participant