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

Yarn Pnp: @n1ru4l/graphql-live-query tried to access tslib #967

Open
simPod opened this issue Aug 11, 2022 · 5 comments · May be fixed by #984
Open

Yarn Pnp: @n1ru4l/graphql-live-query tried to access tslib #967

simPod opened this issue Aug 11, 2022 · 5 comments · May be fixed by #984

Comments

@simPod
Copy link

simPod commented Aug 11, 2022

Getting this error:

Error: @n1ru4l/graphql-live-query tried to access tslib, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

when using yarn (v3.2) Pnp.

with

"@n1ru4l/graphql-live-query@npm:^0.10.0":
  version: 0.10.0
  resolution: "@n1ru4l/graphql-live-query@npm:0.10.0"
  peerDependencies:
    graphql: ^15.4.0 || ^16.0.0
  checksum: c50414f56f9aa7bc60cf73847498fe94dc220f069343a4b8a3feb4f94f588649ab5e9c713e45a7d9c930c0d4337d861fbdc971350b90147eec9215503bb8aeaa
  languageName: node
  linkType: hard
@n1ru4l
Copy link
Owner

n1ru4l commented Aug 12, 2022

@simPod Do you know how to resolve this? I never used pnp or have intentions of using it.

@simPod
Copy link
Author

simPod commented Aug 12, 2022

I believe tslib must be added to the dependencies (example doug-martin/nestjs-query@308db2f) since cjs/index.js requires it const tslib_1 = require("tslib");

@ChrisZieba
Copy link

@n1ru4l If I open a PR adding tslib to the deps does that sound acceptable?

@n1ru4l
Copy link
Owner

n1ru4l commented Aug 23, 2022

@ChrisZieba Yes!

@ChrisZieba
Copy link

#972

@dstaley dstaley linked a pull request Jan 26, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants