Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

graphql-mini-transforms - Use stripIgnoredCharacters to minimize the source (Instead of bespoke regex) #2686

Open
2 tasks done
steven-hoffman-jomashop opened this issue Sep 12, 2023 · 0 comments
Labels
Type: Enhancement 📈 Enhancement to our codebase

Comments

@steven-hoffman-jomashop

Overview

graphql-mini-transforms - Use graphql's stripIgnoredCharacters to minimize the source (Instead of bespoke regex)

Motivation

graphql version 14.3.0 Added the 'stripIgnoredCharacters utility function'
This can be used instead of the bespoke regex used in minifySource
This change will remove some bespoke regex and use a more 'standard' minimization function.

Notes

  • Current minimum dependency for graphql is >=14.50
  • Many documents will not have their hash change; some will have their hash change.

Scope

  • Package: graphql-mini-transforms

Checklist

  • Please delete the labels section before submitting your issue
  • I have described this issue in a way that is actionable (if possible)
@steven-hoffman-jomashop steven-hoffman-jomashop added the Type: Enhancement 📈 Enhancement to our codebase label Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement 📈 Enhancement to our codebase
Projects
None yet
Development

No branches or pull requests

1 participant