Skip to content

[make:entity] Autocompletion for enum classes#1785

Merged
GromNaN merged 14 commits intosymfony:1.xfrom
tcoch:autocompletion-for-enums
Mar 16, 2026
Merged

[make:entity] Autocompletion for enum classes#1785
GromNaN merged 14 commits intosymfony:1.xfrom
tcoch:autocompletion-for-enums

Conversation

@tcoch
Copy link
Copy Markdown
Contributor

@tcoch tcoch commented Feb 13, 2026

This PR aims to provide autocompletion for enum classes, just like it exists for doctrine relations.

Two tests on generating entity with enum types already exist (https://github.com/symfony/maker-bundle/blob/1.x/tests/Maker/MakeEntityTest.php#L697-L741) to cover that this new feature won't break anything.

Copy link
Copy Markdown
Member

@GromNaN GromNaN left a comment

Choose a reason for hiding this comment

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

This feature is interesting, could you add some tests?

@tcoch
Copy link
Copy Markdown
Contributor Author

tcoch commented Mar 5, 2026

This feature is interesting, could you add some tests?

Resolved with aaa64b3

Copy link
Copy Markdown
Member

@GromNaN GromNaN left a comment

Choose a reason for hiding this comment

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

Great!
There are few adjustments to comply with our BC promise.

GromNaN and others added 5 commits March 16, 2026 10:49
@GromNaN GromNaN force-pushed the autocompletion-for-enums branch from 593aa48 to 56c662c Compare March 16, 2026 15:12
Copy link
Copy Markdown
Member

@GromNaN GromNaN left a comment

Choose a reason for hiding this comment

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

The test failure is unrelated to this PR. It's caused by symfony/symfony#62967

@GromNaN
Copy link
Copy Markdown
Member

GromNaN commented Mar 16, 2026

Thank you @tcoch.

@GromNaN GromNaN merged commit 05d2f6c into symfony:1.x Mar 16, 2026
5 of 10 checks passed
GromNaN added a commit that referenced this pull request Mar 17, 2026
 (tcoch)

This PR was squashed before being merged into the 1.x branch.

Discussion
----------

Update Yaml indentation in tests after symfony/symfony#62967

symfony/symfony#62967 introduced a change in how YAML files are generated.
For this reason, some tests are broken here (e.g. #1785).
This PR aims to resolve that.

The modifications are:
- Test files used have been adapted towards the new format
- Special case has been added to `YamlSourceManipulatorTest.php` to address specify of tests in this CI with PHP8.1 and SF6.4.* lowest deps

Commits
-------

ce97fb4 Update Yaml indentation in tests after symfony/symfony#62967
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.

2 participants