Skip to content

feat(semantic): add "unambiguous" SourceType#5529

Closed
Boshen wants to merge 1 commit intomainfrom
upgrade-script
Closed

feat(semantic): add "unambiguous" SourceType#5529
Boshen wants to merge 1 commit intomainfrom
upgrade-script

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Sep 6, 2024

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 6, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@github-actions github-actions bot added A-semantic Area - Semantic A-transformer Area - Transformer / Transpiler labels Sep 6, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Sep 6, 2024

CodSpeed Performance Report

Merging #5529 will not alter performance

Comparing upgrade-script (b2d5716) with main (09aa86d)

Summary

✅ 29 untouched benchmarks

@Boshen Boshen force-pushed the upgrade-script branch 2 times, most recently from 7a57a1a to e1b43ad Compare September 6, 2024 10:51
@magic-akari
Copy link
Contributor

I added a test case from Babel 😈

Expect to Parse: tasks/coverage/babel/packages/babel-parser/test/fixtures/core/sourcetype-unambiguous/import-meta/input.js

× Unexpected import.meta expression
╭─[babel/packages/babel-parser/test/fixtures/core/sourcetype-unambiguous/import-meta/input.js:1:13]
Copy link
Member Author

Choose a reason for hiding this comment

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

Need sawUnambiguousESM from babel parrser

Expect to Parse: tasks/coverage/typescript/tests/cases/compiler/arrayFromAsync.ts

× Expected `(` but found `await`
╭─[typescript/tests/cases/compiler/arrayFromAsync.ts:22:5]
Copy link
Member Author

Choose a reason for hiding this comment

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

Need to check for syntax later.

· ──────
4 │
× Private identifier '#bar' is not allowed outside class bodies
╭─[typescript/tests/cases/conformance/salsa/privateIdentifierExpando.ts:2:3]
Copy link
Member Author

Choose a reason for hiding this comment

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

False postivie.

@Boshen Boshen marked this pull request as ready for review September 6, 2024 14:52
@Boshen Boshen marked this pull request as draft September 6, 2024 14:53
@Boshen
Copy link
Member Author

Boshen commented Sep 6, 2024

I'll try and fix the issues in a stack.

@Boshen
Copy link
Member Author

Boshen commented Sep 7, 2024

Reworked this in the stack #5557

@Boshen Boshen closed this Sep 7, 2024
@Boshen Boshen deleted the upgrade-script branch September 7, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-semantic Area - Semantic A-transformer Area - Transformer / Transpiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants