Skip to content

Conversation

@devversion
Copy link
Member

Fixed up version of #29798

@devversion devversion requested a review from alan-agius4 March 13, 2025 08:09
@devversion devversion added the target: minor This PR is targeted for the next minor release label Mar 13, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Mar 13, 2025
harness.expectFile('dist/browser/main.js.map').content.toContain('/common/index.ts');
harness
.expectFile('dist/browser/main.js.map')
.content.toContain('/core/src/application/application_ref.ts');
Copy link
Member Author

Choose a reason for hiding this comment

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

I count that as an improvement in our source maps, now that we bundle FESM in a single rollup pass :)

@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Mar 13, 2025
@alan-agius4 alan-agius4 merged commit 32470e6 into angular:main Mar 13, 2025
44 checks passed
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants