Skip to content

fix(@schematics/angular): remove explicit index option from new applications #30192

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

clydin
Copy link
Member

@clydin clydin commented Apr 28, 2025

With the index option now defaulting to <project_source_root>/index.html for the application build system, the explicit value present in new applications is no longer required. This removal further reduces the size of the angular.json file for new projects.

Blocked on a release of components.

@clydin clydin added the target: major This PR is targeted for the next major release label Apr 28, 2025
@clydin clydin force-pushed the ng-new/default-index branch 3 times, most recently from ad5d672 to 4968791 Compare April 29, 2025 00:39
@clydin clydin added this to the v20 Candidates milestone Apr 30, 2025
@clydin clydin marked this pull request as ready for review April 30, 2025 15:00
@clydin clydin added target: rc This PR is targeted for the next release-candidate and removed target: major This PR is targeted for the next major release labels Apr 30, 2025
clydin added 2 commits April 30, 2025 19:52
…cations

With the index option now defaulting to `<project_source_root>/index.html`
for the `application` build system, the explicit value present in new
applications is no longer required. This removal further reduces the size
of the `angular.json` file for new projects.
The `@angular/pwa` add schematic now attempts to discover a default index
option value if a configuration usage path within the build target is
possible. The default value as per the `application` build system is
`<project_source_root>/index.html`.
@clydin clydin force-pushed the ng-new/default-index branch from 4968791 to d655307 Compare April 30, 2025 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant