Skip to content

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.

@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", 
@bmish bmish force-pushed the eslint-plugin-eslint-plugin-v4 branch from 5331d62 to 6275be9 Compare October 16, 2021 16:49
@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

3 participants