Skip to content
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

Adopt type=module #50

Merged
merged 7 commits into from
Jun 5, 2021
Merged

Adopt type=module #50

merged 7 commits into from
Jun 5, 2021

Conversation

Fil
Copy link
Member

@Fil Fil commented Apr 27, 2021

follow changes in d3-format:

  • type=module
  • add exports
  • remove zip
  • license: ISC
  • update dependencies

TODO:

  • update README.md
  • check LICENSE
  • depends on d3-array@3
  • switch the default branch to main

Q: am I correct in README that using skypack means we don't need to explicitely import d3-array?

Fil added 2 commits April 27, 2021 06:09
follow changes in d3-format:
* type=module
* add exports
* remove zip
* license: ISC
* update dependencies
@Fil Fil requested a review from mbostock April 27, 2021 04:16
@mbostock
Copy link
Member

Q: am I correct in README that using skypack means we don't need to explicitely import d3-array?

Yep!

@mbostock mbostock merged commit 4447d01 into main Jun 5, 2021
@mbostock mbostock deleted the fil/type-module branch June 5, 2021 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants