Skip to content

fix(compiler): handle trailing comma in object literal#49535

Closed
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:49534/parser-trailing-comma
Closed

fix(compiler): handle trailing comma in object literal#49535
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:49534/parser-trailing-comma

Conversation

@crisbeto
Copy link
Member

Fixes that the compiler wasn't parsing an object literal with a trailing comma correctly.

Fixes #49534.

Fixes that the compiler wasn't parsing an object literal with a trailing comma correctly.

Fixes angular#49534.
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release area: compiler Issues related to `ngc`, Angular's template compiler labels Mar 22, 2023
@ngbot ngbot bot added this to the Backlog milestone Mar 22, 2023
@crisbeto crisbeto marked this pull request as ready for review March 22, 2023 10:23
@pullapprove pullapprove bot requested a review from atscott March 22, 2023 10:23
@atscott atscott removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 22, 2023
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Mar 23, 2023
@atscott
Copy link
Contributor

atscott commented Mar 23, 2023

This PR was merged into the repository by commit 73d2f3c.

atscott pushed a commit that referenced this pull request Mar 23, 2023
Fixes that the compiler wasn't parsing an object literal with a trailing comma correctly.

Fixes #49534.

PR Close #49535
@atscott atscott closed this in 73d2f3c Mar 23, 2023
crapStone pushed a commit to Calciumdibromid/CaBr2 that referenced this pull request Mar 30, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@angular/animations](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fanimations/15.2.4/15.2.5) |
| [@angular/common](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fcommon/15.2.4/15.2.5) |
| [@angular/compiler](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fcompiler/15.2.4/15.2.5) |
| [@angular/compiler-cli](https://github.com/angular/angular/tree/main/packages/compiler-cli) ([source](https://github.com/angular/angular)) | devDependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fcompiler-cli/15.2.4/15.2.5) |
| [@angular/core](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fcore/15.2.4/15.2.5) |
| [@angular/forms](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fforms/15.2.4/15.2.5) |
| [@angular/platform-browser](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fplatform-browser/15.2.4/15.2.5) |
| [@angular/platform-browser-dynamic](https://github.com/angular/angular) | dependencies | patch | [`15.2.4` -> `15.2.5`](https://renovatebot.com/diffs/npm/@angular%2fplatform-browser-dynamic/15.2.4/15.2.5) |

---

### Release Notes

<details>
<summary>angular/angular</summary>

### [`v15.2.5`](https://github.com/angular/angular/blob/HEAD/CHANGELOG.md#&#8203;1525-2023-03-29)

[Compare Source](angular/angular@15.2.4...15.2.5)

##### common

| Commit | Type | Description |
| -- | -- | -- |
| [ca5acadb78](angular/angular@ca5acad) | fix | invalid ImageKit transformation ([#&#8203;49201](angular/angular#49201)) |

##### compiler

| Commit | Type | Description |
| -- | -- | -- |
| [077f6b4674](angular/angular@077f6b4) | fix | do not unquote CSS values ([#&#8203;49460](angular/angular#49460)) |
| [c3cff35869](angular/angular@c3cff35) | fix | handle trailing comma in object literal ([#&#8203;49535](angular/angular#49535)) |

##### core

| Commit | Type | Description |
| -- | -- | -- |
| [d201fc2dec](angular/angular@d201fc2) | fix | set style property value to empty string instead of an invalid value ([#&#8203;49460](angular/angular#49460)) |

##### router

| Commit | Type | Description |
| -- | -- | -- |
| [978d37f324](angular/angular@978d37f) | fix | Ensure Router preloading works with lazy component and static children ([#&#8203;49571](angular/angular#49571)) |
| [a844435514](angular/angular@a844435) | fix | fix [#&#8203;49457](angular/angular#49457) outlet activating with old info ([#&#8203;49459](angular/angular#49459)) |

#### Special Thanks

Alan Agius, Andrew Scott, Asaf Malin, Jan Cabadaj, Kristiyan Kostadinov, Matthieu Riegler, Paul Gschwendtner, Sid and Tano Abeleyra

<!-- CHANGELOG SPLIT MARKER -->

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these updates again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4yNC41IiwidXBkYXRlZEluVmVyIjoiMzUuMjQuNSJ9-->

Co-authored-by: cabr2-bot <cabr2.help@gmail.com>
Reviewed-on: https://codeberg.org/Calciumdibromid/CaBr2/pulls/1842
Reviewed-by: Epsilon_02 <epsilon_02@noreply.codeberg.org>
Co-authored-by: Calciumdibromid Bot <cabr2_bot@noreply.codeberg.org>
Co-committed-by: Calciumdibromid Bot <cabr2_bot@noreply.codeberg.org>
@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 Apr 23, 2023
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Trailing comma in objects in template

3 participants