Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

馃悰 [js-parser] extends infer T extends syntax fails to compile #3934

Closed
1 task done
Conaclos opened this issue Dec 4, 2022 · 1 comment
Closed
1 task done

馃悰 [js-parser] extends infer T extends syntax fails to compile #3934

Conaclos opened this issue Dec 4, 2022 · 1 comment
Labels
A-Parser Area: parser L-TypeScript Area: TypeScript support in Rome S-Bug: confirmed Status: report has been confirmed as a valid bug
Milestone
v12.0.0

Comments

@Conaclos
Copy link
Contributor

Conaclos commented Dec 4, 2022 •
edited

Environment information

Rome latest.

What happened?

Rome amits an error for the following snippet of code:

type B = A extends infer T extends B ? C<T> : never;

Playground

This is also a ESbuild bug recently reported.

Expected result

The code should be parsed without any error.

Code of Conduct

  • I agree to follow Rome's Code of Conduct
@Conaclos Conaclos added the S-To triage Status: user report of a possible bug that needs to be triaged label Dec 4, 2022
@Conaclos Conaclos changed the title 馃悰 [js-parseer] extends infer T extends syntax fails to compile 馃悰 [js-parser] extends infer T extends syntax fails to compile Dec 4, 2022
@ematipico ematipico added S-Bug: confirmed Status: report has been confirmed as a valid bug L-TypeScript Area: TypeScript support in Rome A-Parser Area: parser and removed S-To triage Status: user report of a possible bug that needs to be triaged labels Dec 5, 2022
@nissy-dev
Copy link
Contributor

I think this issue is fixed by #4018

@ematipico ematipico added this to the Next milestone Jan 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Parser Area: parser L-TypeScript Area: TypeScript support in Rome S-Bug: confirmed Status: report has been confirmed as a valid bug
Projects
Rome 2022
Status: Done
Milestone
v12.0.0
Development

No branches or pull requests
3 participants
@ematipico @Conaclos @nissy-dev