Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Oct 23, 2025

This commit addresses an issue where the standalone application schematic did not correctly handle the fileNameStyleGuide: '2016' option. The templateUrl and styleUrl properties in the generated app.component.ts did not include the .component suffix, leading to broken links.

The key changes are:

  • Updates the standalone component template to use the suffix variable for templateUrl and styleUrl, ensuring the correct file paths are generated.
  • Adds a new test case to verify that the generated component has the correct paths when fileNameStyleGuide: '2016' is used.

Closes #31581

…mponents

This commit addresses an issue where the standalone application schematic did not correctly handle the `fileNameStyleGuide: '2016'` option. The `templateUrl` and `styleUrl` properties in the generated `app.component.ts` did not include the `.component` suffix, leading to broken links.

The key changes are:
- Updates the standalone component template to use the `suffix` variable for `templateUrl` and `styleUrl`, ensuring the correct file paths are generated.
- Adds a new test case to verify that the generated component has the correct paths when `fileNameStyleGuide: '2016'` is used.
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Oct 23, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 23, 2025
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 23, 2025
@clydin clydin merged commit 49f12fd into angular:main Oct 23, 2025
35 checks passed
@clydin
Copy link
Member Author

clydin commented Oct 23, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the ng-new/style-guide-file-fixes branch October 23, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @schematics/angular target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[main.ts] import of App is incorrect when file-name-style-guide=2016 is used

2 participants