Skip to content

Conversation

jonathan-meier
Copy link

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Enabling fast type declaration emission is controlled by the flag _geminiAllowEmitDeclarationOnly.

What is the new behavior?

Renamed the flag _geminiAllowEmitDeclarationOnly to _experimentalAllowEmitDeclarationOnly as a follow-up on a comment in PR #61334.

Does this PR introduce a breaking change?

  • Yes
  • No

@pullapprove pullapprove bot requested a review from JoostK May 14, 2025 22:53
@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label May 14, 2025
@ngbot ngbot bot added this to the Backlog milestone May 14, 2025
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

Reviewed-for: public-api

@AndrewKushnir AndrewKushnir added target: minor This PR is targeted for the next minor release action: presubmit The PR is in need of a google3 presubmit labels May 14, 2025
@AndrewKushnir
Copy link
Contributor

Presubmit.

@AndrewKushnir AndrewKushnir added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label May 16, 2025
@AndrewKushnir
Copy link
Contributor

AndrewKushnir commented May 16, 2025

Caretaker note: presubmit is "green".

// FYI, pushed a rebase to restart the CI (it didn't start automatically for some reasons).

…ssion

Rename the flag `_geminiAllowEmitDeclarationOnly` to
`_experimentalAllowEmitDeclarationOnly` as a follow-up on a comment in
PR angular#61334.
@AndrewKushnir AndrewKushnir force-pushed the fix_experimental_allow_declaration_only_flag_name_b416702881 branch from a6c962b to 0e9689d Compare May 16, 2025 23:10
Copy link
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

reviewed-for: fw-general, public-api

@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker and removed action: presubmit The PR is in need of a google3 presubmit labels May 19, 2025
@thePunderWoman
Copy link
Contributor

This PR was merged into the repository by commit 77fa204.

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 Jun 19, 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants