Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Oct 15, 2024

To improve type-checking and reduce hard to diagnose errors when using the new production/development conditions, the set of conditions used by the bundler are now also synchronized with those used by TypeScript. This helps ensure that type mismatches are avoided by type-checking against the actual file that will be bundled into the output.

@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer target: major This PR is targeted for the next major release labels Oct 15, 2024
@clydin clydin force-pushed the application/sync-conditions branch from 3b198f2 to 2700b17 Compare October 15, 2024 11:42
…dler and TypeScript

To improve type-checking and reduce hard to diagnose errors when using the new
`production`/`development` conditions, the set of conditions used by the bundler
are now also synchronized with those used by TypeScript. This helps ensure
that type mismatches are avoided by type-checking against the actual file that
will be bundled into the output.
@clydin clydin force-pushed the application/sync-conditions branch from 2700b17 to 98a89af Compare October 15, 2024 14:19
@clydin clydin requested a review from jkrems October 15, 2024 14:51
Copy link
Contributor

@jkrems jkrems left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@jkrems jkrems 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 15, 2024
@jkrems jkrems merged commit af52fb4 into angular:main Oct 15, 2024
31 checks passed
@clydin clydin deleted the application/sync-conditions branch October 15, 2024 17:26
@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 Nov 15, 2024
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: @angular/build target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants