Skip to content

feat: updated to Typescript 6.0#64

Open
dragomirtitian wants to merge 2 commits into
bloomberg:mainfrom
dragomirtitian:typescript-6.0
Open

feat: updated to Typescript 6.0#64
dragomirtitian wants to merge 2 commits into
bloomberg:mainfrom
dragomirtitian:typescript-6.0

Conversation

@dragomirtitian

Copy link
Copy Markdown
Contributor

Updated to Typescript 6.0

Signed-off-by: Titian Cernicova-Dragomir <tcernicovad1@bloomberg.net>
Comment thread .github/workflows/ci.yml
Comment on lines 18 to 19
- "18.x"
- "20.x"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We ought to have LTS versions here

Comment thread package.json Outdated
Comment thread src/parser.ts
Comment on lines +414 to +417
case ts.SyntaxKind.NoSubstitutionTemplateLiteral:
case ts.SyntaxKind.NumericLiteral:
case ts.SyntaxKind.PrivateIdentifier:
case ts.SyntaxKind.BigIntLiteral:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What's the context behind this change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ts.PropertyName now also includes ts.SyntaxKind.NoSubstitutionTemplateLiteral and ts.SyntaxKind.BigIntLiteral. This is invalid syntax, so probably was added for error handling. Added a test to exercise the number literal case, the NoSubstitutionTemplateLiteral does not seem toa actually be produced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants