Anonymous View
Skip to content

ADT-449-1: The new build is rather large#39

Merged
jemmyw merged 6 commits into
mainfrom
ADT-449-1-shrink-the-build
Mar 16, 2023
Merged

ADT-449-1: The new build is rather large#39
jemmyw merged 6 commits into
mainfrom
ADT-449-1-shrink-the-build

Conversation

@jemmyw

@jemmyw jemmyw commented Mar 16, 2023

Copy link
Copy Markdown
Contributor

Shrink the built file by removing as many external dependencies as possible:

  • Add test for extractReferenceFromName
  • Remove inflected, we only used it to titleize PR state
  • Use string graphql queries so that graphql print does not need importing
  • Use fetch directly instead of using @octokit/graphql
  • Remove moment by replacing the single function it uses

This takes the built gz file from 289kb to 114kb

@jemmyw jemmyw merged commit 68252c1 into main Mar 16, 2023
@jemmyw jemmyw deleted the ADT-449-1-shrink-the-build branch March 16, 2023 20:31
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