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.

Sign up for GitHub

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

Jump to bottom

Require cycle warning #398

Closed
brunomartinezciompi opened this issue Sep 1, 2021 · 2 comments
Closed

Require cycle warning #398

brunomartinezciompi opened this issue Sep 1, 2021 · 2 comments

Comments

@brunomartinezciompi
Copy link

Hey! I am getting this warning on the latest release.

Require cycle: node_modules/semver/classes/comparator.js -> node_modules/semver/classes/range.js -> node_modules/semver/classes/comparator.js

I understand require cycles are allowed and this is just a warning, but it can result in uninitialized values, so it would be nice to get that fixed.
@RayVR
Copy link

RayVR commented Jan 24, 2022

require cycles are technically allowed but should not be tolerated.

@lukekarrys
Copy link
Contributor

This is a duplicate of #318 and #381. I'm keeping #381 open as it has the most information about the issue.

@lukekarrys lukekarrys mentioned this issue Oct 27, 2022
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@RayVR @lukekarrys @brunomartinezciompi