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

Bump minimum requirements & supports SF 7.0 #157

Closed
wants to merge 1 commit into from

Conversation

odolbeau
Copy link
Owner

@odolbeau odolbeau commented Nov 29, 2023

  • Remove support for unmaintained PHP & Symfony versions
  • Support sf7.0

Tests failed for sf7.0 with dev dependencies (prophecy is probably not ready yet do deal with last debal verison? 🤔 ) and for php8.3, should I remove them or we can keep them for later?

IMO this PR should be merged after #154 & released in a new minor version.

Close #143

@odolbeau odolbeau marked this pull request as draft November 29, 2023 12:59
@odolbeau odolbeau force-pushed the bump-dependencies branch 6 times, most recently from e8af8be to 85f6393 Compare November 29, 2023 13:33
@odolbeau odolbeau marked this pull request as ready for review November 29, 2023 13:35
@@ -7,7 +7,7 @@ jobs:
steps:
- uses: actions/checkout@master
- name: PHPStan
uses: docker://jakzal/phpqa:php8.0
uses: docker://jakzal/phpqa:php8.2
Copy link
Owner Author

Choose a reason for hiding this comment

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

8.3 image is not ready yet but it should not be long. :)
jakzal/phpqa#417

@odolbeau odolbeau force-pushed the bump-dependencies branch 4 times, most recently from e6cd314 to 7f94bbd Compare November 29, 2023 13:59
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