Skip to content

Comments

feat(semantic): check for JSDoc types in TS type annotations#6042

Merged
graphite-app[bot] merged 1 commit intomainfrom
don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations
Sep 25, 2024
Merged

feat(semantic): check for JSDoc types in TS type annotations#6042
graphite-app[bot] merged 1 commit intomainfrom
don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations

Conversation

@DonIsaac
Copy link
Contributor

Closes #5982

Copy link
Contributor Author

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

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

@github-actions github-actions bot added the A-semantic Area - Semantic label Sep 24, 2024
@DonIsaac DonIsaac marked this pull request as ready for review September 24, 2024 19:28
@codspeed-hq
Copy link

codspeed-hq bot commented Sep 24, 2024

CodSpeed Performance Report

Merging #6042 will not alter performance

Comparing don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations (8b2e9aa) with main (93575cd)

Summary

✅ 29 untouched benchmarks

@DonIsaac DonIsaac force-pushed the don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations branch from f350cb4 to 25dc8d3 Compare September 24, 2024 20:00
@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Sep 25, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 25, 2024

Merge activity

  • Sep 24, 10:23 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Sep 24, 10:23 PM EDT: Boshen added this pull request to the Graphite merge queue.
  • Sep 24, 10:42 PM EDT: Boshen merged this pull request with the Graphite merge queue.

@Boshen Boshen force-pushed the don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations branch from 25dc8d3 to 8b2e9aa Compare September 25, 2024 02:36
@graphite-app graphite-app bot merged commit 8b2e9aa into main Sep 25, 2024
@graphite-app graphite-app bot deleted the don/09-24-feat_semantic_check_for_jsdoc_types_in_ts_type_annotations branch September 25, 2024 02:42
Boshen added a commit that referenced this pull request Sep 27, 2024
## [0.30.2] - 2024-09-27

### Features

- 60c52ba ast: Allow passing span to `void_0` method (#6065) (Dunqing)
- cca433f codegen: Print `vite` / `webpack` special comments (#6021)
(Dunqing)
- 8d026e1 regular_expression: Implement `GetSpan` for RegExp AST nodes
(#6056) (camchenry)
- 7764793 regular_expression: Implement visitor pattern trait for regex
AST (#6055) (camchenry)
- f866781 semantic: Check for type annotations on left side of `for..in`
and `for..of` iterators (#6043) (DonIsaac)
- 8b2e9aa semantic: Check for JSDoc types in TS type annotations (#6042)
(DonIsaac)
- 28da771 transformer: Do not transform `**` with bigint literals
(#6023) (Boshen)

### Bug Fixes

- a88504c diagnostics: Check for terminal when displaying links (#6018)
(Boshen)
- 418ae25 isolated-declarations: Report uninferrable types in arrays
(#6084) (michaelm)
- e0a8959 minifier: Compute `void number` as `undefined` (#6028)
(Boshen)
- 0658576 paresr: Do not report missing initializer error in ambient
context (#6020) (Boshen)
- b1af73d semantic: Do not create a `global` symbol for `declare global
{}` (#6040) (DonIsaac)
- c8682e9 semantic,codegen,transformer: Handle definite `!` operator in
variable declarator (#6019) (Boshen)

### Performance

- 6b7d3ed isolated-declarations: Should clone transformed AST rather
than original AST (#6078) (Dunqing)
- 85aff19 transformer: Introduce `Stack` (#6093) (overlookmotel)
- ad4ef31 transformer: Introduce `NonEmptyStack` (#6092) (overlookmotel)

### Documentation

- 3099709 allocator: Document `oxc_allocator` crate (#6037) (DonIsaac)
- d60ceb4 oxc: Add README.md and crate-level docs (#6035) (DonIsaac)
- efabfc8 semantic: Improve doc comments on `Reference` methods (#6076)
(overlookmotel)

### Refactor

- 1fc80d1 ast: Move all ts ast related impl methods to `ast_impl`
(#6015) (Dunqing)
- fe696f0 codegen: Simplify printing annotation comments (#6027)
(Dunqing)
- e60ce50 transformer: Add `SparseStack::with_capacity` method (#6094)
(overlookmotel)
- 1399d2c transformer: Move `SparseStack` definition into folder (#6091)
(overlookmotel)
- 6bd29dd transformer: Add more debug assertions (#6090) (overlookmotel)
- c90b9bf transformer: Rename `SparseStack` methods (#6089)
(overlookmotel)
- 2b380c8 transformer: Remove unsued `self.ctx` (#6022) (Boshen)

### Testing

- 93575cd semantic: Add comprehensive regression test suite (#5976)
(DonIsaac)
- a4cec75 transformer: Enable tests (#6032) (overlookmotel)

---------

Co-authored-by: Boshen <[email protected]>
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

0-merge Merge with Graphite Merge Queue A-semantic Area - Semantic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

parser: ? after a type should produce a parse error

2 participants