Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Upgrade d3 to version 7.1.1 #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Upgrade d3 to version 7.1.1 #22

wants to merge 3 commits into from

Conversation

HarelM
Copy link

@HarelM HarelM commented Oct 9, 2021

I'm not sure if this is enough...
But if so it should resolve #21

@ZeevKatz
Copy link
Owner

Great! thanks for helping with this...
Just one thing, build failed on node version, try to upgrade to ^12 in .travis.yml and push again 🙏

@HarelM
Copy link
Author

HarelM commented Oct 10, 2021

I've updated to node 14 and it still fails due to breaking changes in D3 I guess.
I'll look into it...

@HarelM
Copy link
Author

HarelM commented Oct 10, 2021

I'm not sure I complied the list of exported types correctly, I'm not sure how you complied the original list...
In any case, build now passes.
I've tried to update the project to Angular 12 using npx ng update @angular/cli @angular/core @nrwl/schematics @nrwl/workspace but it failed, I'm not sure why... :-(

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.

Upgrade to d3 6.0
2 participants