Skip to content

Add exports to manage "import" and "require" inclusions#832

Merged
stockiNail merged 1 commit into
chartjs:masterfrom
stockiNail:exports
Jan 9, 2023
Merged

Add exports to manage "import" and "require" inclusions#832
stockiNail merged 1 commit into
chartjs:masterfrom
stockiNail:exports

Conversation

@stockiNail
Copy link
Copy Markdown
Collaborator

Should fix #83. @kurkle @LeeLenaleee can you confirm? My experience on packaging is not so huge.

Comment thread package.json
Copy link
Copy Markdown
Member

@kurkle kurkle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks ok to me

@stockiNail
Copy link
Copy Markdown
Collaborator Author

bug? version 2.1.2 or can it wait minor?

@stockiNail stockiNail marked this pull request as ready for review January 9, 2023 14:35
@stockiNail
Copy link
Copy Markdown
Collaborator Author

if we go to 2.1.2 (in my opinion make sense), can we have a look to PR #826 in order to fix the doc as well?

@stockiNail stockiNail added the bug label Jan 9, 2023
@stockiNail stockiNail added this to the 2.1.2 milestone Jan 9, 2023
@stockiNail stockiNail merged commit 599a463 into chartjs:master Jan 9, 2023
@stockiNail stockiNail deleted the exports branch January 9, 2023 15:06
@kurkle
Copy link
Copy Markdown
Member

kurkle commented Jan 9, 2023

212 seems correct to me

@robbytx
Copy link
Copy Markdown

robbytx commented Jan 11, 2023

This PR appears to resolve #786 for me, as I've just described in a comment on that issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lines working in examples; but not on live code

4 participants