Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cleanup] Component blueprint cleanup #20773

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Oct 6, 2024

This removes support for the --component-structure="classic" and --pod arguments from the component blueprint generator.

Part of RFC 995: emberjs/rfcs#1040

@Windvis Windvis changed the title Rfc 995 component blueprint cleanup [Cleanup] Component blueprint cleanup Oct 6, 2024
this.EMBER_GLIMMER_SET_COMPONENT_TEMPLATE &&
commandOptions.componentStructure === 'flat'
) {
if (this.EMBER_GLIMMER_SET_COMPONENT_TEMPLATE && commandOptions.componentStructure === 'flat') {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the EMBER_GLIMMER_SET_COMPONENT_TEMPLATE code can also be removed but that's unrelated to this PR.

The classic file structure is deprecated.
@Windvis Windvis force-pushed the rfc-995-component-generator-cleanup branch from 0d8919f to 36c9a0f Compare October 6, 2024 12:10
The pod layout is deprecated for components.
@Windvis Windvis force-pushed the rfc-995-component-generator-cleanup branch from 36c9a0f to 36242f3 Compare October 6, 2024 12:19
@ef4 ef4 merged commit 4671d46 into emberjs:main Oct 18, 2024
24 checks passed
@ef4
Copy link
Contributor

ef4 commented Oct 18, 2024

Thanks!

@Windvis Windvis deleted the rfc-995-component-generator-cleanup branch October 18, 2024 18:25
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.

3 participants