Skip to content

Add overrides construct for npm only compliance #378

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: beta
Choose a base branch
from
Open

Add overrides construct for npm only compliance #378

wants to merge 1 commit into from

Conversation

JewelPengin
Copy link

This will allow build chains that are using pure npm and not yarn to take advantage of the security fix for d3-color.

@JewelPengin
Copy link
Author

IMHO the true fix should probably be just to upgrade the d3-scale dependency to the latest of ^4.0.2. It builds fine with that version on my end, however there aren't any tests to verify this. This change at least has parity with the resolutions construct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JewelPengin