graphql-tools icon indicating copy to clipboard operation
graphql-tools copied to clipboard

Remove unnecessary dependeny `replaceall` in `@graphql-tools/prisma-loader`

Open GamerBene19 opened this issue 3 years ago • 0 comments
trafficstars

Is your feature request related to a problem? Please describe. Not really a "problem" per se, but I've noticed that this project has an (imo) unnecessary dependency. @graphql-tools/prisma-loader depends on replaceall.
The 12th edition of ECMA-262 added specification for replaceAll which by now is supported by all major browsers.

I think the best solution is to identify uses of replaceall and replace them with replaceAll. I'll create a PR that implements those changes shortly.

Describe alternatives you've considered

I don't think there is an alternative here (apart from keeping the package obv).

Additional context

None.

GamerBene19 avatar Jul 23 '22 19:07 GamerBene19