Skip to content

Conversation

@DonIsaac
Copy link
Contributor

No description provided.

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 16, 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 the A-linter Area - Linter label Sep 16, 2024
@DonIsaac DonIsaac added the C-bug Category - Bug label Sep 16, 2024 — with Graphite App
@DonIsaac DonIsaac marked this pull request as ready for review September 16, 2024 14:40
@codspeed-hq
Copy link

codspeed-hq bot commented Sep 16, 2024

CodSpeed Performance Report

Merging #5807 will not alter performance

Comparing don/09-16-fix_linter_plugin_prefix_name_for_eslint-plugin-node (858f7af) with main (e04841c)

Summary

✅ 29 untouched benchmarks

@DonIsaac DonIsaac force-pushed the don/09-16-fix_linter_plugin_prefix_name_for_eslint-plugin-node branch from 2e2737b to fcc641e Compare September 16, 2024 15:07
@DonIsaac DonIsaac force-pushed the don/09-16-perf_linter_make_all_rules_share_a_diagnostics_vec branch from 52696ff to 09a4b31 Compare September 16, 2024 22:27
@DonIsaac DonIsaac force-pushed the don/09-16-fix_linter_plugin_prefix_name_for_eslint-plugin-node branch from fcc641e to 2c1cc99 Compare September 16, 2024 22:27
@DonIsaac DonIsaac requested a review from camc314 September 17, 2024 05:17
@Boshen Boshen added the 0-merge Merge with Graphite Merge Queue label Sep 17, 2024 — with Graphite App
@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 17, 2024

Merge activity

@Boshen Boshen force-pushed the don/09-16-perf_linter_make_all_rules_share_a_diagnostics_vec branch from 09a4b31 to 3725d5d Compare September 17, 2024 05:46
@Boshen Boshen force-pushed the don/09-16-fix_linter_plugin_prefix_name_for_eslint-plugin-node branch from 2c1cc99 to 858f7af Compare September 17, 2024 05:46
Base automatically changed from don/09-16-perf_linter_make_all_rules_share_a_diagnostics_vec to main September 17, 2024 05:50
@graphite-app graphite-app bot merged commit 858f7af into main Sep 17, 2024
@graphite-app graphite-app bot deleted the don/09-16-fix_linter_plugin_prefix_name_for_eslint-plugin-node branch September 17, 2024 05:52
@oxc-bot oxc-bot mentioned this pull request Sep 18, 2024
Boshen added a commit that referenced this pull request Sep 18, 2024
## [0.9.6] - 2024-09-18

### Features

- 3bf7b24 linter: Make `typescript/no-duplicate-enum-values` a
`correctness` rule (#5810) (DonIsaac)
- 7799c06 linter/react: Implement `no-danger-with-children` rule (#5420)
(Cam McHenry)

### Bug Fixes

- f942485 linter: Remove all* remaining "Disallow <foo>" messages
(#5812) (DonIsaac)
- b5ad518 linter: Improve diagnostic messages for various lint rules
(#5808) (DonIsaac)
- 858f7af linter: Plugin prefix name for eslint-plugin-node (#5807)
(DonIsaac)
- 737ba1d linter: Fix some cases on ```AssignmentExpression``` for
```unicorn/consistent-function-scoping``` (#5675) (Arian94)
- 148c7a8 linter: Replace bitwise AND (&) with logical AND (&&) in
explici… (#5780) (kaykdm)
- b4ed564 linter/no-unused-vars: Writes to members triggering false
positive (#5744) (Dunqing)
- e9c084a linter/no-unused-vars: False positive when a variable used as
a computed member property (#5722) (Dunqing)

### Performance

- 3725d5d linter: Make all rules share a diagnostics vec (#5806)
(DonIsaac)
- e978567 linter: Shrink size of `DisableDirectives` (#5798) (DonIsaac)
- 1bfa515 linter: Remove redundant clone of diagnostics in context
(#5797) (DonIsaac)
- e413cad linter: Move shared context info to `ContextHost` (#5795)
(DonIsaac)

### Refactor

- 6dd6f7c ast: Change `Comment` struct (#5783) (Boshen)
- 7caae5b codegen: Add `GetSpan` requirement to `Gen` trait (#5772)
(Boshen)
- 026ee6a linter: Decouple module resolution from import plugin (#5829)
(dalaoshu)
- 50834bc linter: Move `override_rule` to `OxlintRules` (#5708)
(DonIsaac)
- a438743 linter: Move `OxlintConfig` to `Oxlintrc` (#5707) (DonIsaac)
- f61e8b5 linter: Impl serde and schemars traits for `LintPlugins`
(#5706) (DonIsaac)
- 20a7861 linter: Shorten `Option` syntax (#5735) (overlookmotel)
- d8b612c oxc_linter: Prefer pass Enum instead of str `no_plus_plus`
(#5730) (IWANABETHATGUY)
- cc0408b semantic: S/AstNodeId/NodeId (#5740) (Boshen)

---------

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-bug Category - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants