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

[Dev Deps] Upgrade eslint-plugin-eslint-plugin to v4 #3105

Merged
merged 1 commit into from
Oct 18, 2021

Conversation

bmish
Copy link
Contributor

@bmish bmish commented Oct 16, 2021

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

I suspect this will fail tests in some of the older eslint/node versions we continue to support.

lib/rules/forbid-prop-types.js Show resolved Hide resolved
lib/rules/jsx-no-useless-fragment.js Outdated Show resolved Hide resolved
@bmish
Copy link
Contributor Author

bmish commented Oct 16, 2021

I suspect this will fail tests in some of the older eslint/node versions we continue to support.

Isn't that why we're using ||?

"eslint-plugin-eslint-plugin": "^2.3.0 || ^3.5.3 || ^4.0.1",

@ljharb
Copy link
Member

ljharb commented Oct 16, 2021

That just allows it to be installed. I’m referring to relying on these functions being available on source code instead of context.

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2021

Codecov Report

Merging #3105 (6275be9) into master (210865e) will increase coverage by 0.00%.
The diff coverage is 71.42%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3105   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         118      118           
  Lines        7855     7857    +2     
  Branches     2801     2803    +2     
=======================================
+ Hits         7663     7665    +2     
  Misses        192      192           
Impacted Files Coverage Δ
lib/rules/jsx-sort-default-props.js 84.05% <0.00%> (ø)
lib/rules/jsx-curly-spacing.js 94.11% <66.66%> (ø)
lib/rules/forbid-prop-types.js 100.00% <100.00%> (ø)

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 210865e...6275be9. Read the comment docs.

@ljharb ljharb force-pushed the eslint-plugin-eslint-plugin-v4 branch from 6275be9 to d06f27b Compare October 18, 2021 06:38
@ljharb ljharb changed the title Upgrade eslint-plugin-eslint-plugin to v4 [Dev Deps] Upgrade eslint-plugin-eslint-plugin to v4 Oct 18, 2021
@ljharb ljharb merged commit d06f27b into jsx-eslint:master Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants