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

allow symfony/http-foundation and symfony/validator 5.1 #6344

Merged
merged 2 commits into from Aug 31, 2020

Conversation

dmaicher
Copy link
Contributor

@dmaicher dmaicher commented Aug 28, 2020

Subject

I propose to allow symfony/http-foundation and symfony/validator 5.1.

I am targeting this branch, because its BC and would be nice to have 3.x fully support symfony 5.1 soon.

Changelog

### Added
- support for symfony/http-foundation and symfony/validator 5.1

@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2020

Codecov Report

Merging #6344 into 3.x will increase coverage by 0.00%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff            @@
##                3.x    #6344   +/-   ##
=========================================
  Coverage     77.50%   77.50%           
- Complexity     2603     2605    +2     
=========================================
  Files           142      142           
  Lines          7779     7785    +6     
=========================================
+ Hits           6029     6034    +5     
- Misses         1750     1751    +1     
Impacted Files Coverage Δ Complexity Δ
src/Controller/CRUDController.php 92.81% <75.00%> (-0.12%) 173.00 <0.00> (+1.00) ⬇️
src/Action/RetrieveAutocompleteItemsAction.php 64.64% <100.00%> (ø) 25.00 <0.00> (ø)
src/Admin/AbstractAdmin.php 79.78% <100.00%> (+0.05%) 464.00 <0.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 63de836...b6158da. Read the comment docs.

@dmaicher dmaicher marked this pull request as draft August 28, 2020 19:42
@dmaicher dmaicher force-pushed the more_symfony_5_compat branch 2 times, most recently from 3760963 to 05588e6 Compare August 28, 2020 20:08
@dmaicher dmaicher marked this pull request as ready for review August 28, 2020 20:12
@phansys phansys added the minor label Aug 29, 2020
phansys
phansys previously approved these changes Aug 29, 2020
franmomu
franmomu previously approved these changes Aug 30, 2020
@jordisala1991
Copy link
Member

You will need to rebase again

@jordisala1991 jordisala1991 reopened this Aug 31, 2020
@jordisala1991 jordisala1991 merged commit 51ca7b7 into sonata-project:3.x Aug 31, 2020
@jordisala1991
Copy link
Member

Thank you @dmaicher

@dmaicher dmaicher deleted the more_symfony_5_compat branch September 22, 2020 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants