Skip to content

Conversation

OskarStark
Copy link
Contributor

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

@carsonbot carsonbot added Status: Needs Review Platform Issues & PRs about the AI Platform component labels Sep 24, 2025
@OskarStark OskarStark merged commit a086203 into symfony:main Sep 24, 2025
16 checks passed
&& \is_array($options['beta_features'])
&& !empty($options['beta_features'])
) {
if (isset($options['beta_features']) && \is_array($options['beta_features']) && !empty($options['beta_features'])) {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if (isset($options['beta_features']) && \is_array($options['beta_features']) && !empty($options['beta_features'])) {
if (isset($options['beta_features']) && \is_array($options['beta_features']) && $options['beta_features']) {

speaking about code style :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, will check the empty stuff, just saw the linebreaks 😊

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants