-
Notifications
You must be signed in to change notification settings - Fork 610
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
chore: fix d3-color version to 3.1.0 #2002
Conversation
Flamegraph.com reportCreated by Flamegraph.com Github Action |
Codecov Report
@@ Coverage Diff @@
## main #2002 +/- ##
=======================================
Coverage 60.43% 60.43%
=======================================
Files 177 177
Lines 4999 4999
Branches 1158 1158
=======================================
Hits 3021 3021
Misses 1966 1966
Partials 12 12 |
size-limit report 📦
|
e51cc21
to
b119526
Compare
Flamegraph.com reportCreated by Flamegraph.com Github Action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
@@ -8879,7 +8879,7 @@ d3-chord@3: | |||
d3-path "1 - 3" | |||
|
|||
"d3-color@1 - 3", d3-color@3: | |||
version "3.0.1" | |||
version "3.1.0" | |||
resolved "https://registry.yarnpkg.com/d3-color/-/d3-color-3.0.1.tgz#03316e595955d1fcd39d9f3610ad41bb90194d0a" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Is d3-color a transitive dependency?
- does the version in this
resolved
url need updating?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you use the resolutions
block in the package.json
to codify this change? Just concerned that a different package.json change will re-resolve this back to the old version.
No description provided.