Skip to content

feature branch: downgrade to Symfony 7.4 LTS#244

Open
COil wants to merge 1 commit intomainfrom
symfony-7-4
Open

feature branch: downgrade to Symfony 7.4 LTS#244
COil wants to merge 1 commit intomainfrom
symfony-7-4

Conversation

@COil
Copy link
Copy Markdown
Member

@COil COil commented Feb 7, 2026

Q A
Branch? main
Cleanup? yes
Bug fix? no
New feature? yes
Doc added? yes
Tests pass? yes
Deprecations? yes, see the deprecations.log file.

@COil COil self-assigned this Feb 7, 2026
Copilot AI review requested due to automatic review settings February 7, 2026 12:49
Copy link
Copy Markdown

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

Updates the project dependencies and supporting code to run on Symfony 7.4 LTS (and a lower PHP baseline).

Changes:

  • Downgraded Symfony packages from 8.0 to 7.4 and adjusted PHP requirement to >=8.2.
  • Refactored ListUsersAction to delegate data fetching to a dedicated data service returning a DTO, and updated the Twig template accordingly.
  • Added eraseCredentials() to User and exercised it in the unit test; updated internal docs to mention the new Data/ directory.

Reviewed changes

Copilot reviewed 8 out of 11 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
tests/Unit/Entity/UserTest.php Calls eraseCredentials() during entity test.
templates/App/Controller/ListUsersAction.html.twig Switches view inputs to data.usersDbal / data.usersOrm and updates displayed example code.
src/Entity/User.php Adds eraseCredentials() (with an #[\Override] attribute).
src/Data/Controller/ListUsersActionData.php Introduces data service assembling DBAL + ORM data into a DTO.
src/Controller/ListUsersActionDto.php Introduces DTO for ListUsersAction view data.
src/Controller/ListUsersAction.php Injects data service and renders it as data.
composer.json Downgrades Symfony constraints to 7.4 and lowers PHP requirement to >=8.2.
AGENTS.md Documents the new src/Data/ directory.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@COil COil changed the title chore: Downgrade to Symfony 7.4 LTS feature branch: downgrade to Symfony 7.4 LTS Feb 7, 2026
@COil COil added the feature branch Feeature branch demo label Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature branch Feeature branch demo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants