Skip to content

Update symfony support#10

Merged
MKodde merged 6 commits intomainfrom
feature/update-symfony-support
Nov 13, 2023
Merged

Update symfony support#10
MKodde merged 6 commits intomainfrom
feature/update-symfony-support

Conversation

@MKodde
Copy link
Copy Markdown
Member

@MKodde MKodde commented Oct 31, 2023

For the new version of the Bundle (4.0.0) we aim to drop SF 4.4. support and start allowing SF 7 use

I also dropped support for PHP 7 and <8.2.

@MKodde MKodde force-pushed the feature/update-symfony-support branch from aadb6c4 to 1496323 Compare October 31, 2023 09:31
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 31, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c0017c9) 41.17% compared to head (5aab82c) 41.17%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #10   +/-   ##
=========================================
  Coverage     41.17%   41.17%           
  Complexity       52       52           
=========================================
  Files            12       12           
  Lines           153      153           
=========================================
  Hits             63       63           
  Misses           90       90           
Files Coverage Δ
src/OpenConextMonitorBundle.php 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MKodde MKodde force-pushed the feature/update-symfony-support branch from 1496323 to 4d2e2d7 Compare October 31, 2023 09:41
@MKodde MKodde force-pushed the feature/update-symfony-support branch from 4d2e2d7 to e97bbfc Compare October 31, 2023 09:42
@Chris53897
Copy link
Copy Markdown
Contributor

I have created a PR to support Symfony 7.
malukenho/docheader#43

Comment thread composer.json Outdated
"phpunit/phpunit": "^9.6|^10.4",
"sebastian/phpcpd": "^4.1|^5.0|^6.0",
"squizlabs/php_codesniffer": "^3.6",
"malukenho/docheader": "^0.1.8",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can be changed to "malukenho/docheader": "^0.1.8|^1.0",
But needs check for BC breaks.

This will fix the failing composer install. https://github.com/OpenConext/Monitor-bundle/actions/runs/6704694822

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion @Chris53897

I'll be updating and testing this right away

@MKodde MKodde merged commit b73d0a4 into main Nov 13, 2023
@MKodde MKodde deleted the feature/update-symfony-support branch November 13, 2023 09:31
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