Skip to content

fix(@angular/build): allow a default application browser option #30240

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

Merged

Conversation

clydin
Copy link
Member

@clydin clydin commented May 4, 2025

The application build system's browser option is now considered optional. If not present, the value will be an main.ts file within the configured project source root (sourceRoot). This change allows the removal of the browser option from any project that uses the default generated value.

The application build system's `browser` option is now considered optional.
If not present, the value will be an `main.ts` file within the configured
project source root (`sourceRoot`). This change allows the removal of the
`browser` option from any project that uses the default generated value.
@clydin clydin force-pushed the application/allow-default-browser-option branch from 7a403d0 to e775654 Compare May 4, 2025 23:35
@clydin clydin marked this pull request as ready for review May 5, 2025 01:21
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label May 5, 2025
@clydin clydin requested a review from alan-agius4 May 5, 2025 13:24
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label May 5, 2025
@clydin clydin merged commit c560d44 into angular:main May 5, 2025
33 checks passed
@clydin
Copy link
Member Author

clydin commented May 5, 2025

The changes were merged into the following branches: main, 20.0.x

@clydin clydin deleted the application/allow-default-browser-option branch May 5, 2025 14:56
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: @angular/build target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants