Skip to content

refactor(compiler): Remove the interpolation config#64071

Closed
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:remove-interpolation-confg
Closed

refactor(compiler): Remove the interpolation config#64071
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:remove-interpolation-confg

Conversation

@JeanMeche
Copy link
Copy Markdown
Member

@JeanMeche JeanMeche commented Sep 25, 2025

After #63474, we don't need that anymore.

@JeanMeche JeanMeche requested a review from crisbeto September 25, 2025 09:21
@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label Sep 25, 2025
@ngbot ngbot bot added this to the Backlog milestone Sep 25, 2025
@JeanMeche JeanMeche force-pushed the remove-interpolation-confg branch from 3a67d2f to 058e6fa Compare September 25, 2025 09:45
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Sep 25, 2025
@JeanMeche JeanMeche force-pushed the remove-interpolation-confg branch from 058e6fa to 89d437b Compare September 25, 2025 21:19
@JeanMeche JeanMeche force-pushed the remove-interpolation-confg branch from 89d437b to 8d01326 Compare September 26, 2025 11:43
JeanMeche
JeanMeche commented Sep 26, 2025
View reviewed changes
packages/compiler/src/ml_parser/defaults.ts
}

/**
* This symbol is referenced inside G3 and will require some cleanup.
Copy link
Copy Markdown
Member Author

@JeanMeche JeanMeche Sep 26, 2025

Choose a reason for hiding this comment

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

Callout. Remove this will require a g3 cleanup.

@kirjs
Copy link
Copy Markdown
Contributor

kirjs commented Sep 26, 2025

This PR was merged into the repository. The changes were merged into the following branches:

kirjs pushed a commit that referenced this pull request Sep 26, 2025
@JeanMeche @kirjs
refactor(compiler): Remove the interpolation config (#64071)
9dbcff8 
After #63474, we don't need that anymore.

PR Close #64071
@kirjs kirjs closed this in 768a09d Sep 26, 2025
@JeanMeche JeanMeche reopened this Sep 26, 2025
@thePunderWoman thePunderWoman added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker labels Sep 26, 2025
thePunderWoman added a commit to thePunderWoman/angular that referenced this pull request Sep 26, 2025
@thePunderWoman
Revert "refactor(compiler): Remove the interpolation config (angular#…
4a98c35 
…64071)"

This reverts commit 768a09d.
kirjs added a commit to kirjs/angular that referenced this pull request Sep 26, 2025
@kirjs
Revert "refactor(compiler): Remove the interpolation config (angular#…
83c273e 
…64071)"

This reverts commit 768a09d.
kirjs pushed a commit that referenced this pull request Sep 26, 2025
@thePunderWoman @kirjs
Revert "refactor(compiler): Remove the interpolation config (#64071)" (…
d900dba 
…#64110)

This reverts commit 768a09d.

PR Close #64110
kirjs pushed a commit that referenced this pull request Sep 26, 2025
@thePunderWoman @kirjs
Revert "refactor(compiler): Remove the interpolation config (#64071)" (…
221d568 
…#64110)

This reverts commit 768a09d.

PR Close #64110
@JeanMeche JeanMeche added target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release labels Sep 28, 2025
@JeanMeche
Copy link
Copy Markdown
Member Author

JeanMeche commented Sep 29, 2025 •
edited
Loading

caretaker note: please sync with this CL cl/812576532

@JeanMeche JeanMeche added merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Sep 29, 2025
@kirjs
Copy link
Copy Markdown
Contributor

kirjs commented Sep 29, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@kirjs kirjs closed this in 04462ed Sep 29, 2025
@angular-automatic-lock-bot
Copy link
Copy Markdown

angular-automatic-lock-bot bot commented Oct 30, 2025

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 Oct 30, 2025
inesane pushed a commit to inesane/angular that referenced this pull request Mar 3, 2026
@JeanMeche @inesane
refactor(compiler): Remove the interpolation config (angular#64071)
df8884b 
After angular#63474, we don't need that anymore.

PR Close angular#64071
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@crisbeto crisbeto crisbeto approved these changes

Assignees

No one assigned

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: major This PR is targeted for the next major release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

4 participants

@JeanMeche @kirjs @crisbeto @thePunderWoman