Skip to content

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Oct 16, 2024

Includes the following changes to fix the broken CI:

fix(compiler): handle typeof expressions in serializer

Fixes that some changes which landed at the same time caused a compilation error in the serialized.

test(compiler-cli): fix broken test

Fixes a test that broke because a pipe wasn't marked explicitly as standalone: false.

test(compiler): update failing tests

Fixes some tests that started failing, because #58154 made it so placeholder-only messages are extracted while #58176 added some tests that only contain placeholders.

Fixes that some changes which landed at the same time caused a compilation error in the serialized.
@crisbeto crisbeto 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 16, 2024
@crisbeto crisbeto requested a review from devversion October 16, 2024 08:02
@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label Oct 16, 2024
@ngbot ngbot bot added this to the Backlog milestone Oct 16, 2024
@devversion devversion 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 16, 2024
@angular-robot angular-robot bot added area: compiler Issues related to `ngc`, Angular's template compiler and removed area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 16, 2024
@crisbeto crisbeto changed the title fix(compiler): handle typeof expressions in serializer Fix CI failures Oct 16, 2024
@angular-robot angular-robot bot added area: compiler Issues related to `ngc`, Angular's template compiler and removed area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 16, 2024
@ngbot ngbot bot modified the milestone: Backlog Oct 16, 2024
Fixes a test that broke because a pipe wasn't marked explicitly as `standalone: false`.
Fixes some tests that started failing, because angular#58154 made it so placeholder-only messages are extracted while angular#58176 added some tests that only contain placeholders.
@crisbeto crisbeto force-pushed the typeof-compilation-error branch from 37fd227 to 26d6aab Compare October 16, 2024 08:46
@angular-robot angular-robot bot added area: compiler Issues related to `ngc`, Angular's template compiler and removed area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 16, 2024
@devversion
Copy link
Member

Thanks @crisbeto

@devversion
Copy link
Member

This PR was merged into the repository by commit 5971037.

The changes were merged into the following branches: main

devversion pushed a commit that referenced this pull request Oct 16, 2024
Fixes a test that broke because a pipe wasn't marked explicitly as `standalone: false`.

PR Close #58217
devversion pushed a commit that referenced this pull request Oct 16, 2024
Fixes some tests that started failing, because #58154 made it so placeholder-only messages are extracted while #58176 added some tests that only contain placeholders.

PR Close #58217
@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 16, 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: compiler Issues related to `ngc`, Angular's template compiler 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