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

Warnings during service:push in Node v14.8.0 with the CLI #2100

Open
tvvignesh opened this issue Aug 29, 2020 · 1 comment
Open

Warnings during service:push in Node v14.8.0 with the CLI #2100

tvvignesh opened this issue Aug 29, 2020 · 1 comment
Labels
🔨 cli related to the CLI itself

Comments

@tvvignesh
Copy link

tvvignesh commented Aug 29, 2020

Intended outcome:

I was pushing the schema using service:push to apollo studio using npx apollo service:push --graph=tc-graph --key=service:tokenhere --variant=current --serviceName=svc-name --serviceURL=http://localhost:port/ --localSchemaFile=./path/to/GQLFile.gql and it was supposed to work without any warnings.

Actual outcome:

Screenshot 2020-08-29 145601

It throws varning like this.

How to reproduce the issue:

I guess this has been happening with other packages as well since Node 14.x (ref: nodejs/node#32987) and this is the PR related to it: nodejs/node#29935

Versions

apollo/2.30.3 linux-x64 node-v14.8.0

@eduhenke
Copy link

eduhenke commented Sep 4, 2020

Can confirm this, if you run with --trace-warnings it tells you that it's coming from the shelljs module:

$ node --trace-warnings `which apollo`  service:push --graph=GRAPHL --variant=VARIANT --serviceName=SERVICE_NAME --serviceURL=SERVICE_URL --localSchemaFile=/path/to/schema.graphql
(node:132) Warning: Accessing non-existent property 'cat' of module exports inside circular dependency
    at emitCircularRequireWarning (internal/modules/cjs/loader.js:848:11)
    at Object.get (internal/modules/cjs/loader.js:862:5)
    at Object._register [as register] (/usr/local/share/.config/yarn/global/node_modules/shelljs/src/common.js:439:12)
    at Object.<anonymous> (/usr/local/share/.config/yarn/global/node_modules/shelljs/src/cat.js:4:8)
    at Module._compile (internal/modules/cjs/loader.js:1251:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1272:10)
    at Module.load (internal/modules/cjs/loader.js:1100:32)
    at Function.Module._load (internal/modules/cjs/loader.js:962:14)
    at Module.require (internal/modules/cjs/loader.js:1140:19)
    at require (internal/modules/cjs/helpers.js:75:18)
...

@abernix abernix added the 🔨 cli related to the CLI itself label Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 cli related to the CLI itself
Projects
None yet
Development

No branches or pull requests

3 participants