Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs] update react/jsx-runtime link #3160

Merged
merged 1 commit into from Jan 1, 2022
Merged

[Docs] update react/jsx-runtime link #3160

merged 1 commit into from Jan 1, 2022

Conversation

tatsushitoji
Copy link
Contributor

Small issues with react/jsx-runtime docs

README.md Outdated
@@ -31,7 +31,7 @@ Use [our preset](#recommended) to get reasonable defaults:
]
```

If you are using the [new JSX transform from React 17](https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#removing-unused-react-imports), extend [`react/jsx-runtime`](https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/index.js#L163-L176) in your eslint config (add `"plugin:react/jsx-runtime"` to `"extends"`) to disable the relevant rules.
If you are using the [new JSX transform from React 17](https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#removing-unused-react-imports), extend [`react/jsx-runtime`](https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/index.js#L168-L182) in your eslint config (add `"plugin:react/jsx-runtime"` to `"extends"`) to disable the relevant rules.
Copy link
Member

Choose a reason for hiding this comment

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

thanks! let's change this to a commit sha so it doesn't break again in the future.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for your review.
Is this fix correct? db9bda0

@codecov-commenter
Copy link

codecov-commenter commented Dec 31, 2021

Codecov Report

Merging #3160 (ccf3c35) into master (6a40d48) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3160   +/-   ##
=======================================
  Coverage   97.54%   97.54%           
=======================================
  Files         120      120           
  Lines        8265     8265           
  Branches     2975     2975           
=======================================
  Hits         8062     8062           
  Misses        203      203           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a40d48...ccf3c35. Read the comment docs.

@ljharb ljharb merged commit ccf3c35 into jsx-eslint:master Jan 1, 2022
@tatsushitoji tatsushitoji deleted the patch-1 branch January 5, 2022 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants