Skip to content

Comments

feat(parser): fix incorrect AST for x?.f<T>()#7387

Merged
graphite-app[bot] merged 1 commit intomainfrom
11-21-feat_parser_fix_incorrect_ast_for_x_.f_t_
Nov 21, 2024
Merged

feat(parser): fix incorrect AST for x?.f<T>()#7387
graphite-app[bot] merged 1 commit intomainfrom
11-21-feat_parser_fix_incorrect_ast_for_x_.f_t_

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Nov 21, 2024

No description provided.

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 21, 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-linter Area - Linter A-parser Area - Parser A-transformer Area - Transformer / Transpiler labels Nov 21, 2024
Copy link
Member Author

Boshen commented Nov 21, 2024

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

@github-actions github-actions bot added the C-enhancement Category - New feature or request label Nov 21, 2024
@Boshen Boshen marked this pull request as draft November 21, 2024 06:03
@Boshen Boshen marked this pull request as ready for review November 21, 2024 06:10
@Boshen Boshen self-assigned this Nov 21, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 21, 2024

Merge activity

@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Nov 21, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 21, 2024

CodSpeed Performance Report

Merging #7387 will not alter performance

Comparing 11-21-feat_parser_fix_incorrect_ast_for_x_.f_t_ (e6922df) with main (885e37f)

Summary

✅ 30 untouched benchmarks

@Boshen Boshen force-pushed the 11-21-feat_parser_fix_incorrect_ast_for_x_.f_t_ branch from 39d74b6 to e6922df Compare November 21, 2024 06:10
@graphite-app graphite-app bot merged commit e6922df into main Nov 21, 2024
@graphite-app graphite-app bot deleted the 11-21-feat_parser_fix_incorrect_ast_for_x_.f_t_ branch November 21, 2024 06:15
This was referenced Nov 21, 2024
Boshen added a commit that referenced this pull request Nov 21, 2024
## [0.13.0] - 2024-11-21

- 7bf970a linter: [**BREAKING**] Remove tree_shaking plugin (#7372)
(Boshen)
- 7f8747d linter: Implement `react/no-array-index-key` (#6960)
(BitterGourd)

### Features

- be152c0 linter: Add `typescript/no-require-imports` rule (#7315)
(Dmitry Zakharov)
- 849489e linter: Add suggestion for no-console (#4312) (DonIsaac)
- 8cebdc8 linter: Allow appending plugins in override (#7379)
(camchenry)
- 8cfea3c oxc_cfg: Add implicit return instruction (#5568)
(IWANABETHATGUY)
- e6922df parser: Fix incorrect AST for `x?.f<T>()` (#7387) (Boshen)

### Bug Fixes

- e91c287 linter: Fix panic in react/no-array-index-key (#7395) (Boshen)
- a32f5a7 linter/no-array-index-key: Compile error due to it uses a
renamed API (#7391) (Dunqing)
- 666b6c1 parser: Add missing `ChainExpression` in optional
`TSInstantiationExpression` (#7371) (Boshen)

### Documentation

- df143ca linter: Add docs for config settings (#4827) (DonIsaac)
- ad44cfa linter: Import/first options (#7381) (Zak)

### Refactor

- c34d649 linter: Use `scope_id` etc methods (#7394) (overlookmotel)
- 466f395 vscode: Split `ConfigService` and `Config` (#7376) (Alexander
S.)

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

0-merge Merge with Graphite Merge Queue A-linter Area - Linter A-parser Area - Parser A-transformer Area - Transformer / Transpiler C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant