Skip to content

Conversation

@DjordyKoert
Copy link
Collaborator

@DjordyKoert DjordyKoert commented Nov 14, 2025

Description

Closes #2590

What type of PR is this? (check all applicable)

  • Bug Fix
  • Feature
  • Refactor
  • Deprecation
  • Breaking Change
  • Documentation Update
  • CI

Checklist

  • I have made corresponding changes to the documentation (docs/)
  • I have made corresponding changes to the changelog (CHANGELOG.md)

Copilot AI review requested due to automatic review settings November 14, 2025 13:36
Copilot finished reviewing on behalf of DjordyKoert November 14, 2025 13:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds CI test configurations for Symfony 7.4 and 8.0 versions, extending the test matrix to include these newer framework versions with PHP 8.2, 8.3, and 8.4.

Key Changes:

  • Adds 6 new test matrix combinations: Symfony 7.4 with PHP 8.2/8.3/8.4 and Symfony 8.0 with PHP 8.2/8.3/8.4
  • Follows existing pattern of testing all supported PHP versions against each Symfony version

@DjordyKoert DjordyKoert force-pushed the symfony-7.4-and-8 branch 2 times, most recently from f867d56 to 58c3a62 Compare November 14, 2025 13:59
@codecov
Copy link

codecov bot commented Nov 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.60%. Comparing base (95b2057) to head (2ea25ef).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##              5.x    #2612   +/-   ##
=======================================
  Coverage   96.60%   96.60%           
=======================================
  Files          94       94           
  Lines        3034     3034           
=======================================
  Hits         2931     2931           
  Misses        103      103           

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dmaicher

This comment was marked as outdated.

@dmaicher
Copy link

And

   Problem 3
    - Root composer.json requires willdurand/hateoas-bundle ^2.7 -> satisfiable by willdurand/hateoas-bundle[2.7.0].
    - willdurand/hateoas-bundle 2.7.0 requires symfony/expression-language ~3.0 || ~4.0 || ~5.0 || ~6.0 || ~7.0 -> found symfony/expression-language[v3.0.0, ..., v3.4.47, v4.0.0-RC1, ..., v4.4.47, v5.0.0-RC1, ..., v5.4.45, v6.0.0-RC1, ..., v6.4.24, v7.0.0-RC1, ..., v7.4.0-RC1] but these were not loaded, likely because it conflicts with another require.

needs willdurand/Hateoas#343 first and then we can look into willdurand/BazingaHateoasBundle#118

@DjordyKoert DjordyKoert mentioned this pull request Nov 28, 2025
8 tasks
@DjordyKoert DjordyKoert changed the title ci(symfony): run on 7.4 and 8.0 chore(symfony): support 8.0 Nov 28, 2025
DjordyKoert added a commit that referenced this pull request Dec 4, 2025
## Description

Run optional-dependencies. This should help with determining if a major
version bump of Symfony (8) is safe to do without needing to wait for
external bundles to update their version constraints (or build support)

Related to #2612

## What type of PR is this? (check all applicable)
- [ ] Bug Fix
- [ ] Feature
- [ ] Refactor
- [ ] Deprecation
- [ ] Breaking Change
- [ ] Documentation Update
- [x] CI

## Checklist
- [ ] I have made corresponding changes to the documentation (`docs/`)

---------

Co-authored-by: Copilot <[email protected]>
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.

Support Symfony 8

3 participants