Skip to content

fix(napi/transform): remove confusing jsx option#6159

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

fix(napi/transform): remove confusing jsx option#6159
graphite-app[bot] merged 1 commit into
mainfrom
09-29-fix_napi_transform_remove_unused_jsx_option

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 force-pushed the 09-29-fix_napi_transform_remove_unused_jsx_option branch from 56b91cd to f27d59f Compare September 29, 2024 04:09
@graphite-app

graphite-app Bot commented Sep 29, 2024

Copy link
Copy Markdown
Contributor

Merge activity

@graphite-app graphite-app Bot merged commit f27d59f into main Sep 29, 2024
@graphite-app graphite-app Bot deleted the 09-29-fix_napi_transform_remove_unused_jsx_option branch September 29, 2024 04:16
@codspeed-hq

codspeed-hq Bot commented Sep 29, 2024

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #6159 will not alter performance

Comparing 09-29-fix_napi_transform_remove_unused_jsx_option (f27d59f) with main (ea908f7)

Summary

✅ 29 untouched 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.

1 participant