Skip to content
This repository was archived by the owner on Jun 16, 2025. It is now read-only.

Conversation

rumpl
Copy link
Member

@rumpl rumpl commented Nov 8, 2020

We cheat a little bit here, if we see a react with version >=17 we
immediately say that it is used because rules of how and when this is
true are too complicated to implement with the system we have.

Fixes #591

@rumpl rumpl force-pushed the feat-react17 branch 3 times, most recently from fd8aa8e to b42d3b0 Compare November 8, 2020 11:20
@codecov
Copy link

codecov bot commented Nov 8, 2020

Codecov Report

Merging #594 (7d24db9) into main (9875fd7) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #594      +/-   ##
==========================================
+ Coverage   98.57%   98.58%   +0.01%     
==========================================
  Files          47       48       +1     
  Lines         980      988       +8     
==========================================
+ Hits          966      974       +8     
  Misses         14       14              
Impacted Files Coverage Δ
src/special/react17.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 84c6af4...7d24db9. Read the comment docs.

We cheat a little bit here, if we see a react with version >=17 we
immediately say that it is used because rules of how and when this is
true are too complicated to implement with the system we have.

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl rumpl merged commit bc8146a into main Nov 8, 2020
@rumpl rumpl deleted the feat-react17 branch November 8, 2020 11:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

false alert for React 17
1 participant