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

Specify required version of tslib ^2.0.0 #31

Merged
merged 1 commit into from
Aug 9, 2021

Conversation

efokschaner
Copy link
Contributor

The version + features of typescript used to compile this lib means that tslib needs to be at version 2.0.0 minimum.

Because antvis/algorithm doesn't specify an explicit version, it is not guaranteed that developers receive the necessary version.
This causes the following error at import time:

./node_modules/@antv/algorithm/es/gaddi.js
Attempted import error: '__spreadArray' is not exported from 'tslib'.

You can see here all the other projects updating their tslib dependencies similarly in the references to this PR: microsoft/tslib#133

This relates to
antvis/G6#2702
and
antvis/G6#2713
it is likely necessary, but may not be sufficient, to fix those reported issues. There could be other antvis packages with the same issue.

The version + features of typescript used to compile this lib means that tslib needs to be at version 2.0.0 minimum.

Because `antvis/algorithm` doesn't specify an explicit version, it is not guaranteed that developers receive the necessary version.
This causes the following error at import time:
```
./node_modules/@antv/algorithm/es/gaddi.js
Attempted import error: '__spreadArray' is not exported from 'tslib'.
```

You can see here all the other projects updating their tslib dependencies similarly in the references to this PR: microsoft/tslib#133

This relates to
antvis/G6#2702
and
antvis/G6#2713
it is likely necessary, but may not be sufficient, to fix those reported issues. There could be other `antvis` packages with the same issue.
@Yanyan-Wang Yanyan-Wang merged commit 9319488 into antvis:master Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants