Skip to content

Comments

feat(linter): do not bail for unmatched rules yet#7093

Merged
Boshen merged 1 commit intomainfrom
11-03-feat_linter_do_not_bail_for_unmatched_rules_yet
Nov 3, 2024
Merged

feat(linter): do not bail for unmatched rules yet#7093
Boshen merged 1 commit intomainfrom
11-03-feat_linter_do_not_bail_for_unmatched_rules_yet

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Nov 3, 2024

I intend to make a release today, let's hold onto this behaviour.

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 3, 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-cli Area - CLI C-enhancement Category - New feature or request labels Nov 3, 2024
@Boshen Boshen added 0-merge Merge with Graphite Merge Queue and removed C-enhancement Category - New feature or request labels Nov 3, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 3, 2024

CodSpeed Performance Report

Merging #7093 will not alter performance

Comparing 11-03-feat_linter_do_not_bail_for_unmatched_rules_yet (af6b8d5) with main (79bf74a)

Summary

✅ 30 untouched benchmarks

Copy link
Member Author

Boshen commented Nov 3, 2024

Merge activity

  • Nov 2, 10:40 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.

@Boshen Boshen force-pushed the 11-03-feat_linter_do_not_bail_for_unmatched_rules_yet branch from 1432d4a to af6b8d5 Compare November 3, 2024 03:13
@github-actions github-actions bot added A-editor Area - Editor and Language Server C-enhancement Category - New feature or request labels Nov 3, 2024
@Boshen Boshen merged commit 2184588 into main Nov 3, 2024
@Boshen Boshen deleted the 11-03-feat_linter_do_not_bail_for_unmatched_rules_yet branch November 3, 2024 03:18
@oxc-bot oxc-bot mentioned this pull request Nov 3, 2024
Boshen added a commit that referenced this pull request Nov 3, 2024
## [0.11.0] - 2024-11-03

- 9fd9f4f linter: [**BREAKING**] Sync sindresorhus/globals; removed
Object.prototype properties from builtin and es* globals (#6991)
(Boshen)

### Features

- 6b619da editor: Listen to config file changes and trigger a
didChangeConfiguration update (#6964) (Nicholas Rayburn)
- 7872927 editor/vscode: Support window/showMessage event (#7085)
(Nicholas Rayburn)
- 2184588 linter: Do not bail for unmatched rules yet (#7093) (Boshen)
- a6fcd81 linter: Add `import/no-commonjs` rule (#6978) (Dmitry
Zakharov)
- 1691cab linter: Support user-configurable secrets for
`oxc-security/api-keys` (#5938) (DonIsaac)
- 610621c linter: Implement `react/style-prop-object` (#6342) (Albert
Kaaman)
- 1e2f012 linter: Add `oxc/no-map-spread` (#6751) (DonIsaac)
- 1c66473 linter: Implement `eslint/prefer-object-has-own` (#6905)
(tomoya yanagibashi)

### Bug Fixes

- ebf3753 editor: Fix onConfigChange to send the correct config for
didChangeConfiguration notification (#6962) (Nicholas Rayburn)
- 79bf74a linter: Check is_reference_to_global_variable in
`no-array-constructor` (#7067) (Naoya Yoshizawa)
- 38d1f78 linter: Remove confusing help text for now (#7081) (Cam
McHenry)
- 147e2e4 linter: Allow replacing rule when none are enabled yet (#7014)
(camchenry)
- 7aa496a linter: Remove unsafe fixer of `no-useless-spread` (#6655)
(dalaoshu)
- f5a7134 linter/no-unused-vars: False positive for discarded reads
within sequences (#6907) (DonIsaac)

### Documentation

- 4551baa linter: Document `rules` (#6983) (Boshen)

### Refactor

- a8dc75d linter: Remove unused CLI result types (#7088) (camchenry)
- 8f1460e linter: Move `LintPlugins` from `LintOptions` to `LintConfig`
(#6932) (DonIsaac)

### Testing

- c35d3f2 linter: Improve test failure output (#6975) (camchenry)

---------

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-cli Area - CLI A-editor Area - Editor and Language Server A-linter Area - Linter C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant