Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix CI failures #58217

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

Closed
wants to merge 3 commits into from
Closed

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