Skip to content

Filler: Refactor complex functions - #10039

Closed
kurkle wants to merge 1 commit into
chartjs:masterfrom
kurkle:fill-chore
Closed

kurkle wants to merge 1 commit into
chartjs:masterfrom
kurkle:fill-chore

Conversation

@kurkle

@kurkle kurkle commented Jan 1, 2022

Copy link
Copy Markdown
Member

No description provided.

etimberg
etimberg previously approved these changes Jan 1, 2022
@kurkle
kurkle marked this pull request as draft January 1, 2022 21:07
@kurkle
kurkle marked this pull request as ready for review January 1, 2022 21:44
@kurkle kurkle added this to the Version 3.8.0 milestone Jan 1, 2022
@kurkle

kurkle commented Jan 1, 2022

Copy link
Copy Markdown
Member Author

I'll create a separate PR exploding the filler to multiple files, but will leave this open too, if we decide to keep it in one file.

@kurkle

kurkle commented Jan 1, 2022

Copy link
Copy Markdown
Member Author
Filename Size Change
dist/chart.esm.js 73.7 kB +65 B (0%)
dist/chart.js 93.2 kB +64 B (0%)
dist/chart.min.js 66.2 kB +14 B (0%)

@kurkle

kurkle commented Jan 2, 2022

Copy link
Copy Markdown
Member Author

Closing in favor of #10040

@kurkle kurkle closed this Jan 2, 2022
@kurkle
kurkle deleted the fill-chore branch January 2, 2022 08:10
@kurkle kurkle removed this from the Version 3.8.0 milestone Jan 11, 2022
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