Skip to content

Conversation

@JeanMeche
Copy link
Member

@JeanMeche JeanMeche commented Jan 16, 2025

The diagnostic was created in #49512 but was never added to the list of diagnostic that run.

fixes #59569

Since this PR enables a diagnostic, we'll need a TGP.

@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label Jan 16, 2025
@ngbot ngbot bot added this to the Backlog milestone Jan 16, 2025
@JeanMeche JeanMeche added action: rerun CI at HEAD requires: TGP This PR requires a passing TGP before merging is allowed labels Jan 16, 2025
@ngbot
Copy link

ngbot bot commented Jan 16, 2025

@JeanMeche the CircleCI rerun you requested failed. See details below:

invalid json response body at https://circleci.com/api/v2/project/gh/angular/angular/pipeline?circle-token=0b8e85cdbc076b09bcd8a1f4f3d7c6b7eff2109e reason: Unexpected token I in JSON at position 0

The diagnostic was created in angular#49512 but was never added to the list of diagnostic that run.

fixes angular#59569
@JeanMeche JeanMeche force-pushed the skip-hydration-diagnostic branch from 6a5cca9 to d510256 Compare January 16, 2025 22:14
@JeanMeche JeanMeche added the target: minor This PR is targeted for the next minor release label Jan 16, 2025
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Jan 16, 2025
@JeanMeche JeanMeche marked this pull request as ready for review January 17, 2025 15:48
@pullapprove pullapprove bot requested a review from kirjs January 17, 2025 15:48
@pullapprove pullapprove bot removed the requires: TGP This PR requires a passing TGP before merging is allowed label Jan 17, 2025
@kirjs kirjs added the action: merge The PR is ready for merge by the caretaker label Jan 17, 2025
@JeanMeche JeanMeche removed the action: merge The PR is ready for merge by the caretaker label Jan 17, 2025
@JeanMeche
Copy link
Member Author

@kirjs I think we should run a TGP for this first as it introduces a new extended diagnostic that could break existing targets.

@JeanMeche JeanMeche added the requires: TGP This PR requires a passing TGP before merging is allowed label Jan 18, 2025
@pullapprove pullapprove bot removed the requires: TGP This PR requires a passing TGP before merging is allowed label Jan 18, 2025
@JeanMeche JeanMeche modified the milestones: Backlog, v 19.2 candidates Jan 21, 2025
@AndrewKushnir AndrewKushnir changed the title feature(compiler): Add Skip Hydration diagnostic. feat(compiler): Add Skip Hydration diagnostic. Feb 14, 2025
@thePunderWoman thePunderWoman added the action: merge The PR is ready for merge by the caretaker label Feb 14, 2025
@thePunderWoman
Copy link
Contributor

TESTED=TGP

@thePunderWoman
Copy link
Contributor

TGP is green. This is safe to merge.

@thePunderWoman
Copy link
Contributor

This PR was merged into the repository by commit 5b20bab.

The changes were merged into the following branches: main

@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 Mar 21, 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: compiler Issues related to `ngc`, Angular's template compiler detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TS-994005: Angular compiler option "extendedDiagnostics.checks" has an unknown check: "skipHydrationNotStatic".

3 participants