Skip to content

Conversation

OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Sep 2, 2025

Q A
Bug fix? no
New feature? no
Docs? no
Issues --
License MIT

Replace all short-circuit evaluation patterns (|| throw) with classic if statements for better readability and consistency across the codebase. AFAIK this is more the Symfony-way of doing.

@OskarStark OskarStark self-assigned this Sep 2, 2025
@carsonbot carsonbot changed the title Refactor short-circuit evaluation to classic if statements Refactor short-circuit evaluation to classic if statements Sep 2, 2025
@OskarStark OskarStark added Platform Issues & PRs about the AI Platform component Store Issues & PRs about the AI Store component labels Sep 2, 2025
@carsonbot carsonbot changed the title Refactor short-circuit evaluation to classic if statements [Platform][Store] Refactor short-circuit evaluation to classic if statements Sep 2, 2025
Replace all short-circuit evaluation patterns (`|| throw`) with classic
if statements for better readability and consistency across the codebase.
@OskarStark OskarStark merged commit b72429a into symfony:main Sep 2, 2025
7 checks passed
This was referenced Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Platform Issues & PRs about the AI Platform component Status: Needs Review Store Issues & PRs about the AI Store component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants