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

[DO NOT MERGE] Use rollup-plugin-visualizer to gather some stats on the file #3109

Closed
wants to merge 10 commits into from

Conversation

romaricpascal
Copy link
Member

@romaricpascal romaricpascal commented Dec 15, 2022

A quick exploration of what rollup-plugin-visualizer could provide for keeping track of performance. This will require us to upgrade to Rollup 3, so is not a now thing. The plugin provides different visual representation of our library's modules:

  • a treemap and a sunburst to highlight the size contributed by each module
  • a network to highlight which modules connect to which. Unfortunately packed a little tightly to see anything.

It also provides two data formats that we could consume in scripts to derive information:

The final commit adds specific file names for each entry being built by rollup. This would help us collect stats about each separate component.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3109 December 15, 2022 17:12 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3109 December 15, 2022 17:17 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3109 December 15, 2022 17:29 Inactive
Each type of chart available is generated in the 'public' folder so it can be served in the review app
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3109 December 15, 2022 17:32 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3109 December 15, 2022 18:06 Inactive
@colinrotherham colinrotherham force-pushed the rollup-standalone branch 2 times, most recently from 1c05ea0 to abdf26b Compare December 16, 2022 17:07
@colinrotherham colinrotherham force-pushed the rollup-standalone branch 6 times, most recently from 721b159 to 7765b4a Compare December 20, 2022 13:51
Base automatically changed from rollup-standalone to main December 20, 2022 14:34
@romaricpascal
Copy link
Member Author

Spike work is complete and perf work carries on in #3279

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants