Skip to content

Conversation

@shulaoda
Copy link
Member

@shulaoda shulaoda commented Sep 28, 2024

closes #3426

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 28, 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.

@shulaoda shulaoda force-pushed the feat/support-local-version-for-vscode-extension branch from 3658c50 to 703e2fe Compare September 28, 2024 03:30
@shulaoda shulaoda marked this pull request as ready for review September 28, 2024 06:26
@shulaoda shulaoda force-pushed the feat/support-local-version-for-vscode-extension branch from 703e2fe to 93448bd Compare September 28, 2024 06:26
@shulaoda shulaoda force-pushed the feat/support-local-version-for-vscode-extension branch from 36d988e to 9db1bfe Compare September 28, 2024 22:32
@shulaoda shulaoda force-pushed the feat/support-local-version-for-vscode-extension branch from 9db1bfe to dd9b847 Compare September 28, 2024 22:33
@shulaoda shulaoda requested a review from DonIsaac September 28, 2024 22:33
@shulaoda shulaoda requested a review from camc314 September 30, 2024 01:21
@Boshen
Copy link
Member

Boshen commented Sep 30, 2024

Need to test.

Did you manage to test this locally?

@shulaoda
Copy link
Member Author

Did you manage to test this locally?

At the beginning of the draft, I conducted testing and it has now been completed.

@DonIsaac DonIsaac added C-enhancement Category - New feature or request A-linter Area - Linter 0-merge Merge with Graphite Merge Queue labels Oct 6, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Oct 6, 2024

Merge activity

  • Oct 5, 8:24 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Oct 5, 8:24 PM EDT: DonIsaac added this pull request to the Graphite merge queue.

@DonIsaac DonIsaac merged commit 1a5f293 into oxc-project:main Oct 6, 2024
@shulaoda shulaoda deleted the feat/support-local-version-for-vscode-extension branch October 6, 2024 01:07
This was referenced Oct 7, 2024
Boshen added a commit that referenced this pull request Oct 8, 2024
## [0.9.10] - 2024-10-07

### Features

- f272137 editors/vscode: Clear diagnostics on file deletion (#6326)
(dalaoshu)
- 1a5f293 editors/vscode: Update VSCode extention to use project's
language server (#6132) (dalaoshu)
- 376cc09 linter: Implement `no-throw-literal` (#6144) (dalaoshu)
- 5957214 linter: Allow fixing in files with source offsets (#6197)
(camchenry)
- a089e19 linter: Eslint/no-else-return (#4305) (yoho)
- 183739f linter: Implement prefer-await-to-callbacks (#6153) (dalaoshu)
- ae539af linter: Implement no-return-assign (#6108) (Radu Baston)

### Bug Fixes

- 9e9808b linter: Fix regression when parsing ts in vue files (#6336)
(Boshen)
- 93c6db6 linter: Improve docs and diagnostics message for
no-else-return (#6327) (DonIsaac)
- e0a3378 linter: Correct false positive in
`unicorn/prefer-string-replace-all` (#6263) (H11)
- ea28ee9 linter: Improve the fixer of `prefer-namespace-keyword`
(#6230) (dalaoshu)
- f6a3450 linter: Get correct source offsets for astro files (#6196)
(camchenry)
- be0030c linter: Allow whitespace control characters in
`no-control-regex` (#6140) (camchenry)
- e7e8ead linter: False positive in `no-return-assign` (#6128)
(DonIsaac)

### Performance

- ac0a82a linter: Reuse allocator when there are multiple source texts
(#6337) (Boshen)
- 50a0029 linter: Do not concat vec in `no-useless-length-check` (#6276)
(camchenry)

### Documentation

- 7ca70dd linter: Add docs for `ContextHost` and `LintContext` (#6272)
(camchenry)
- a949ecb linter: Improve docs for `eslint/getter-return` (#6229)
(DonIsaac)
- 14ba263 linter: Improve docs for `eslint-plugin-import` rules (#6131)
(dalaoshu)

### Refactor

- 642725c linter: Rename vars from `ast_node_id` to `node_id` (#6305)
(overlookmotel)
- 8413175 linter: Move shared function from utils to rule (#6127)
(dalaoshu)
- ba9c372 linter: Make jest/vitest rule mapping more clear (#6273)
(camchenry)
- 82b8f21 linter: Add schemars and serde traits to AllowWarnDeny and
RuleCategories (#6119) (DonIsaac)
- ea908f7 linter: Consolidate file loading logic (#6130) (DonIsaac)
- db751f0 linter: Use regexp AST visitor in `no-control-regex` (#6129)
(camchenry)
- 3aa7e42 linter: Use RegExp AST visitor for `no-hex-escape` (#6117)
(camchenry)
- 9d5b44a linter: Use regex visitor in `no-regex-spaces` (#6063)
(camchenry)
- 0d44cf7 linter: Use regex visitor in `no-useless-escape` (#6062)
(camchenry)
- eeb8873 linter: Use regex visitor in `no-empty-character-class`
(#6058) (camchenry)

### Testing

- d883562 linter: Invalid `eslint/no-unused-vars` options (#6228)
(DonIsaac)

---------

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-linter Area - Linter C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update VSCode extention to use project's language server.

4 participants