Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Dec 19, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

<!-- What / Why -->
<!-- Describe the request in detail. What it does and why it's being
changed. -->
while reading https://docs.npmjs.com/cli/v11/commands/npm-dedupe

<img width="1582" height="502" alt="image"
src="https://github.com/user-attachments/assets/cc8109a3-6d56-40bb-8cc7-09f6c59e0dd8"
/>

I realized there's probably a typo/logic error/imprecision: the
`c@1.9.0` is mentioned nowhere else.

I assume `c@1.9.9` was meant, because it's listed in the dependency
graph above. but I have no clue about the inner workings of
`npm-dedupe`, so I might be wrong here.

---

alternative solution: noting `c@1.9.x`, but not using a range and
explicitly matching the listed version above inherits more clarity, at
least in my mind

## References
<!-- Examples:
  Related to #0
  Depends on #0
  Blocked by #0
  Fixes #0
  Closes #0
-->

- https://docs.npmjs.com/cli/v11/commands/npm-dedupe
@pull pull bot locked and limited conversation to collaborators Dec 19, 2025
@pull pull bot added the ⤵️ pull label Dec 19, 2025
@pull pull bot merged commit 3474ec3 into LadyK-21:latest Dec 19, 2025
5 of 8 checks passed
@LadyK-21
Copy link
Owner

⚠️ Snyk checks are incomplete.

Status Scanner Critical High Medium Low Total (0)
⚠️ Open Source Security 0 0 0 0 See details

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants