Skip to content

Comments

feat(linter): implement react/style-prop-object#6342

Merged
DonIsaac merged 7 commits intooxc-project:mainfrom
nekosaur:feat/style-prop-object-rule
Oct 28, 2024
Merged

feat(linter): implement react/style-prop-object#6342
DonIsaac merged 7 commits intooxc-project:mainfrom
nekosaur:feat/style-prop-object-rule

Conversation

@nekosaur
Copy link
Contributor

@nekosaur nekosaur commented Oct 7, 2024

Implements not recommended rule react/style-prop-object (#1022)

@graphite-app
Copy link
Contributor

graphite-app bot commented Oct 7, 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 Oct 7, 2024
@DonIsaac
Copy link
Contributor

DonIsaac commented Oct 7, 2024

Thanks for your interest in Oxc! This PR will be a great contribution. Just a few small fixes and we should be able to merge this.

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 12, 2024

CodSpeed Performance Report

Merging #6342 will not alter performance

Comparing nekosaur:feat/style-prop-object-rule (536f250) with main (9725e3c)

Summary

✅ 30 untouched benchmarks

@github-actions github-actions bot added the C-enhancement Category - New feature or request label Oct 16, 2024
@DonIsaac DonIsaac merged commit 610621c into oxc-project:main Oct 28, 2024
Orenbek pushed a commit to Orenbek/oxc that referenced this pull request Oct 28, 2024
Implements not recommended rule `react/style-prop-object`
(oxc-project#1022)

---------

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

A-linter Area - Linter C-enhancement Category - New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants