lifeomic / graphql-resolvers-xray-tracing

A GraphQL middleware to enable X-Ray tracing subsegments for GraphQL resolvers
MIT License
68 stars 16 forks source link

-d was unexpected at this time #73

Closed mikemeerschaert closed 3 years ago

mikemeerschaert commented 3 years ago

Hi, 2 members of my team have been having trouble installing this package using git bash, for whatever reason npm running on git bash in windows is complaining about the if statement in the postinstall script. I had both of them try just typing a bash if statement to check for a directory and that seems to work, it's just failing when npm tries to execute the postinstall script.

Looking at that script, it appears that it's checking if there is a tests directory, and if so execute graphql-codegen. Is it necessary to have package consumers run this script? or could it be moved elsewhere?

https://github.com/lifeomic/graphql-resolvers-xray-tracing/blob/e87ff6b1d0d15c27a9097a8ed037d81da2257e2c/package.json#L20

image

FYI, they are both running windows 10, and git bash version is 4.4.12(3) npm version is 6.14.12 node version is 12.22.1

@DavidTanner