Skip to content

Comments

fix(transformer): arrow func transform use correct spans#3630

Merged
graphite-app[bot] merged 1 commit intomainfrom
06-11-fix_transformer_arrow_func_transform_use_correct_spans
Jun 12, 2024
Merged

fix(transformer): arrow func transform use correct spans#3630
graphite-app[bot] merged 1 commit intomainfrom
06-11-fix_transformer_arrow_func_transform_use_correct_spans

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Jun 11, 2024

Arrow function transform generate output with correct spans.

Copy link
Member Author

overlookmotel commented Jun 11, 2024

@github-actions github-actions bot added the A-transformer Area - Transformer / Transpiler label Jun 11, 2024
@overlookmotel overlookmotel marked this pull request as ready for review June 11, 2024 17:14
@codspeed-hq
Copy link

codspeed-hq bot commented Jun 11, 2024

CodSpeed Performance Report

Merging #3630 will not alter performance

Comparing 06-11-fix_transformer_arrow_func_transform_use_correct_spans (5cb7e6a) with main (21d3425)

Summary

✅ 22 untouched benchmarks

@overlookmotel overlookmotel force-pushed the 06-11-fix_transformer_arrow_function_transform_alter__this_ branch from df2b475 to d4bad20 Compare June 11, 2024 17:53
@overlookmotel overlookmotel force-pushed the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch from 7906044 to 0ce0c8f Compare June 11, 2024 17:53
@overlookmotel overlookmotel force-pushed the 06-11-fix_transformer_arrow_function_transform_alter__this_ branch from d4bad20 to a111962 Compare June 11, 2024 20:00
@overlookmotel overlookmotel force-pushed the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch from 0ce0c8f to 6143b5f Compare June 11, 2024 20:00
@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 12, 2024

Merge activity

@Boshen Boshen force-pushed the 06-11-fix_transformer_arrow_function_transform_alter__this_ branch from a111962 to db83da3 Compare June 12, 2024 02:55
Boshen pushed a commit that referenced this pull request Jun 12, 2024
Arrow function transform generate output with correct spans.
@Boshen Boshen force-pushed the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch from 6143b5f to 1dfda46 Compare June 12, 2024 02:56
@Boshen Boshen force-pushed the 06-11-fix_transformer_arrow_function_transform_alter__this_ branch from db83da3 to 0c4ccb4 Compare June 12, 2024 02:59
Boshen pushed a commit that referenced this pull request Jun 12, 2024
Arrow function transform generate output with correct spans.
@Boshen Boshen force-pushed the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch from 1dfda46 to 82efa58 Compare June 12, 2024 03:00
Base automatically changed from 06-11-fix_transformer_arrow_function_transform_alter__this_ to main June 12, 2024 03:07
Arrow function transform generate output with correct spans.
@Boshen Boshen force-pushed the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch from 82efa58 to 5cb7e6a Compare June 12, 2024 04:11
@graphite-app graphite-app bot merged commit 5cb7e6a into main Jun 12, 2024
@graphite-app graphite-app bot deleted the 06-11-fix_transformer_arrow_func_transform_use_correct_spans branch June 12, 2024 04:16
This was referenced Jun 12, 2024
Boshen added a commit that referenced this pull request Jun 12, 2024
## [0.14.0] - 2024-06-12

### Breaking

* fix(codegen)!: remove the unecessary 4th argument from `Codegen::new`
(#3640)
* feat(ast)!: make `Trivias` clonable by adding `Arc` (#3638)

### Features

- f6d9ca6 linter: Add `eslint/sort-imports` rule (#3568) (Wang Wenzhe)
- 129f91e span: Port over more methods from TextRange (#3592) (Don
Isaac)

### Bug Fixes

- f8f6d33 ast: Correct `visited_node` attr for strict mode of arrow fns
(#3635) (overlookmotel)
- e6ad3fb diagnostics: Do not print ansi color codes in non-TTYs (#3624)
(Boshen)
- d65202d span: Correct doc comments (#3608) (overlookmotel)
- 35e267b transformer: Arrow function transform use UIDs for `_this`
vars (#3634) (overlookmotel)
- 39bdebc transformer: Arrow func transform maintain scope ID (#3633)
(overlookmotel)
- 5cb7e6a transformer: Arrow func transform use correct spans (#3630)
(overlookmotel)
- 0c4ccb4 transformer: Arrow function transform alter `</this>` (#3627)
(overlookmotel)
- 8d237c4 transformer: JSX source calculate correct column when Unicode
chars (#3615) (overlookmotel)
- 9e8f4d6 transformer: Do not add `__source` for generated nodes (#3614)
(overlookmotel)
- 0fb4c35 transformer: Use UID for JSX source filename var (#3612)
(overlookmotel)

### Performance

- 3a59294 transformer: React display name transform reduce Atom
allocations (#3616) (overlookmotel)
- f4c1389 transformer: Create `Vec` with capacity (#3613)
(overlookmotel)

### Refactor

- 0f92521 ast: Replace recursion with loop (#3626) (overlookmotel)
- 08f1010 ast: Make `AstBuilder` `Copy` (#3602) (overlookmotel)
- 84304b4 linter: Add a `ctx.module_record()` method (#3637) (Boshen)
- f98f777 linter: Add rule fixer (#3589) (Don Isaac)
- e90e6a2 minifier: Make `Prepass` `Copy` (#3603) (overlookmotel)
- 7d61832 semantic: Pass `Rc` by value (#3586) (overlookmotel)
- 89bcbd5 transformer: Move `BoundIdentifier` into helpers (#3610)
(overlookmotel)
- 5793ff1 transformer: Replace `&’a Trivias` with `Rc<Trivias>` (#3580)
(Dunqing)
- 509871f transformer: Comment for unimplemented `spec` option in arrow
fns transform (#3618) (overlookmotel)
- 4b2e3a7 transformer: Fix indentation (#3617) (overlookmotel)
- 3467e3d transformer: Remove outdated comment (#3606) (overlookmotel)
- a799225 transformer: Flatten file structure for React transform
(#3604) (overlookmotel)
- 70f31a8 transformer: Reduce branching in JSX transform (#3596)
(overlookmotel)
- 3ae567d transformer: Remove dead code (#3588) (overlookmotel)
- 60cbdec traverse: `generate_uid_in_root_scope` method (#3611)
(overlookmotel)

Co-authored-by: Boshen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-transformer Area - Transformer / Transpiler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant