Skip to content

Fix pnpm workspace parsing and udpate package detection - #4079

Merged
AyanSinhaMahapatra merged 2 commits into
developfrom
fix-npm-workspace-parsing
Jan 9, 2025
Merged

Fix pnpm workspace parsing and udpate package detection#4079
AyanSinhaMahapatra merged 2 commits into
developfrom
fix-npm-workspace-parsing

Conversation

@AyanSinhaMahapatra

@AyanSinhaMahapatra AyanSinhaMahapatra commented Jan 8, 2025

Copy link
Copy Markdown
Member

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁
  • Updated documentation pages (if applicable)
  • Updated CHANGELOG.rst (if applicable)

Reference: aboutcode-org/scancode.io#1521
Reported-by: @JonoYang
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
See https://github.com/pnpm/spec/blob/master/lockfile/9.0.md

Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra changed the title Fix pnpm workspace parsing and package detection Fix pnpm workspace parsing and udpate package detection Jan 8, 2025

@JonoYang JonoYang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@AyanSinhaMahapatra LGTM! Thanks for the fix!

@AyanSinhaMahapatra
AyanSinhaMahapatra merged commit 54ee82e into develop Jan 9, 2025
@AyanSinhaMahapatra
AyanSinhaMahapatra deleted the fix-npm-workspace-parsing branch January 9, 2025 10:59
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.

Error when running scan_codebase on npm package using a download url source

2 participants