Skip to content

feat(linter) eslint-plugin-react/jsx-key#1016

Merged
Boshen merged 4 commits into
oxc-project:mainfrom
camc314:c/jsx-key
Oct 20, 2023
Merged

feat(linter) eslint-plugin-react/jsx-key#1016
Boshen merged 4 commits into
oxc-project:mainfrom
camc314:c/jsx-key

Conversation

@camc314
Copy link
Copy Markdown
Contributor

@camc314 camc314 commented Oct 19, 2023

@github-actions github-actions Bot added the A-linter Area - Linter label Oct 19, 2023
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Oct 19, 2023

CodSpeed Performance Report

Merging #1016 will not alter performance

Comparing camc314:c/jsx-key (2af008a) with main (46a5c42)

Summary

✅ 22 untouched benchmarks

@Boshen
Copy link
Copy Markdown
Member

Boshen commented Oct 20, 2023

@camc314 I gave you triage permission so you can update the corresponding issues https://github.com/web-infra-dev/oxc/invitations

You may give me a wave on our discord channel if you want to get more involved https://discord.gg/9uXCAwqQZW.

Comment thread crates/oxc_linter/src/rules/react/jsx_key.rs Outdated
Comment thread crates/oxc_linter/src/rules/react/jsx_key.rs Outdated
Comment thread crates/oxc_linter/src/rules/react/jsx_key.rs Outdated
Copy link
Copy Markdown
Member

@Boshen Boshen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the amazing work! Just a few above questions and we'll get this merged!

Comment thread crates/oxc_linter/src/snapshots/jsx_key.snap Outdated
@github-actions github-actions Bot added the A-ast Area - AST label Oct 20, 2023
@Boshen
Copy link
Copy Markdown
Member

Boshen commented Oct 20, 2023

Let's merge this first and then rebase the other PR.

Seems like there is a clippy error, and some commented out tests.

@camc314 camc314 requested a review from Boshen October 20, 2023 07:09
@Boshen Boshen merged commit 3f06335 into oxc-project:main Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ast Area - AST A-linter Area - Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants