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

Dependency tree fixed #507

Closed
wants to merge 1 commit into from

Conversation

deprov447
Copy link

Updated some of the dependencies manually to fix dependency tree error. Able to run npm update now.
Resolves issue #506

Signed-off-by: Anubhav Choudhary [email protected]

Signed-off-by: Anubhav Choudhary <[email protected]>
@deprov447
Copy link
Author

Also, apollo-upload-server is long deprecated and the replacement graphql-upload can be used in place. But this requires some source code changes also (so maybe a different ticket?)

Copy link
Contributor

@clementkng clementkng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for looking into this! As someone who's not too familiar with resolving these issues, could I ask which npm version are you using and how were you able to determine which packages needed to be updated?

@deprov447
Copy link
Author

I am using npm version: 8.4.0. npm update gave me a ERRESOLVE error with hint on which dependencies are in conflict. IIRC I had to remove whatever version these deps were and manually install them again.

@t-fine
Copy link
Collaborator

t-fine commented Oct 19, 2022

Closing this since we archived the cloud/sdr infrastructure code

@t-fine t-fine closed this Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants