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 tests/code for php 7.4 #32981

Merged
merged 1 commit into from Aug 7, 2019
Merged

Conversation

jderusse
Copy link
Member

@jderusse jderusse commented Aug 6, 2019

Q A
Branch? 3.4
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #32844
License MIT
Doc PR NA

Fix remaining tests and deprecation

@jderusse jderusse changed the base branch from 4.4 to 3.4 August 6, 2019 09:13
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Aug 6, 2019
@jderusse jderusse force-pushed the deprec-remaining branch 8 times, most recently from 914c8b7 to e0dda4f Compare August 6, 2019 13:20
@nicolas-grekas
Copy link
Member

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit 05ec8a0 into symfony:3.4 Aug 7, 2019
nicolas-grekas added a commit that referenced this pull request Aug 7, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

Fix tests/code for php 7.4

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #32844
| License       | MIT
| Doc PR        | NA

Fix remaining tests and deprecation

Commits
-------

05ec8a0 Fix remaining tests
@jderusse jderusse deleted the deprec-remaining branch August 8, 2019 11:36
nicolas-grekas added a commit that referenced this pull request Aug 8, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

Fix deprecations on 4.3

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #32844
| License       | MIT
| Doc PR        | NA

Fix deprecations in branch 4.3
note: remaining deprecation `assertStringContainsString` will be fixed in #32977

* [ ] fix tests in branch 3.4 in #32981

Commits
-------

8fd16a6 Fix deprecation on 4.3
This was referenced Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants