Skip to content

feat(napi/transform): display semantic error#6160

Merged
graphite-app[bot] merged 1 commit into
mainfrom
09-29-feat_napi_transform_display_semantic_error
Sep 29, 2024
Merged

feat(napi/transform): display semantic error#6160
graphite-app[bot] merged 1 commit into
mainfrom
09-29-feat_napi_transform_display_semantic_error

Conversation

@Boshen

@Boshen Boshen commented Sep 29, 2024

Copy link
Copy Markdown
Member

No description provided.

@graphite-app

graphite-app Bot commented Sep 29, 2024

Copy link
Copy Markdown
Contributor

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.

Boshen commented Sep 29, 2024

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on Graphite Graphite

@Boshen Boshen changed the base branch from 09-29-fix_napi_transform_remove_unused_jsx_option to graphite-base/6160 September 29, 2024 04:09
@graphite-app

graphite-app Bot commented Sep 29, 2024

Copy link
Copy Markdown
Contributor

Merge activity

@Boshen Boshen changed the base branch from graphite-base/6160 to main September 29, 2024 04:16
@Boshen Boshen force-pushed the 09-29-feat_napi_transform_display_semantic_error branch from 27aad16 to 15552ac Compare September 29, 2024 04:17
@graphite-app graphite-app Bot merged commit 15552ac into main Sep 29, 2024
@graphite-app graphite-app Bot deleted the 09-29-feat_napi_transform_display_semantic_error branch September 29, 2024 04:23
@codspeed-hq

codspeed-hq Bot commented Sep 29, 2024

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #6160 will create unknown performance changes

Comparing 09-29-feat_napi_transform_display_semantic_error (15552ac) with main (f27d59f)

Summary

🆕 29 new benchmarks

Benchmarks breakdown

Benchmark main 09-29-feat_napi_transform_display_semantic_error Change
🆕 codegen[checker.ts] N/A 21.2 ms N/A
🆕 codegen_sourcemap[checker.ts] N/A 72.9 ms N/A
🆕 isolated-declarations[vue-id.ts] N/A 397 ms N/A
🆕 lexer[RadixUIAdoptionSection.jsx] N/A 23.7 µs N/A
🆕 lexer[antd.js] N/A 21.8 ms N/A
🆕 lexer[cal.com.tsx] N/A 5.4 ms N/A
🆕 lexer[checker.ts] N/A 13 ms N/A
🆕 lexer[pdf.mjs] N/A 3.5 ms N/A
🆕 linter[cal.com.tsx] N/A 1.3 s N/A
🆕 linter[checker.ts] N/A 2.4 s N/A
🆕 minifier[antd.js] N/A 596 ms N/A
🆕 minifier[react.development.js] N/A 5.2 ms N/A
🆕 minifier[typescript.js] N/A 894.5 ms N/A
🆕 parser[RadixUIAdoptionSection.jsx] N/A 79.6 µs N/A
🆕 parser[antd.js] N/A 107.3 ms N/A
🆕 parser[cal.com.tsx] N/A 24.7 ms N/A
🆕 parser[checker.ts] N/A 53.3 ms N/A
🆕 parser[pdf.mjs] N/A 17.5 ms N/A
🆕 semantic[RadixUIAdoptionSection.jsx] N/A 100.2 µs N/A
🆕 semantic[antd.js] N/A 118.8 ms N/A
... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

Boshen added a commit that referenced this pull request Sep 29, 2024
## [0.30.5] - 2024-09-29

### Features

- 15552ac napi/transform: Display semantic error (#6160) (Boshen)
- f50fdcd napi/transform: Make react refresh option take a boolean
(#6146) (Boshen)

### Bug Fixes

- f27d59f napi/transform: Remove confusing `jsx` option (#6159) (Boshen)
- bfd1988 transformer/react: Should not collect use-hooks if it's a
nested member expression (#6143) (Dunqing)

### Refactor

- ab187d1 codegen: Restrict visibility of internal methods (#6145)
(DonIsaac)
- 375bebe transformer: Improve parsing React pragmas (#6138)
(overlookmotel)
- 0836f6b transformer: Move parsing pragmas into TS transform (#6137)
(overlookmotel)
- 30424fa transformer: TS transforms only store options they need
(#6135) (overlookmotel)

---------

Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
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