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

🧹 refactor npm parser #3328

Merged
merged 2 commits into from
Feb 15, 2024
Merged

🧹 refactor npm parser #3328

merged 2 commits into from
Feb 15, 2024

Conversation

chris-rock
Copy link
Member

This refactors the npm package parser. It does not expose it to mql yet.

Copy link
Contributor

github-actions bot commented Feb 15, 2024

Test Results

2 773 tests  +28   2 772 ✅ +28   45s ⏱️ -1s
  186 suites ± 0       1 💤 ± 0 
    5 files   ± 0       0 ❌ ± 0 

Results for commit 804eb64. ± Comparison against base commit d6aaab8.

♻️ This comment has been updated with latest results.

@chris-rock chris-rock force-pushed the chris-rock/npm-resources branch 3 times, most recently from 3572527 to a820c50 Compare February 15, 2024 16:21
providers/os/resources/npm/package_json.go Outdated Show resolved Hide resolved
providers/os/resources/npm/package_json.go Show resolved Hide resolved
providers/os/resources/npm/package_json.go Show resolved Hide resolved
@chris-rock chris-rock force-pushed the chris-rock/npm-resources branch from a820c50 to 804eb64 Compare February 15, 2024 18:50
@chris-rock chris-rock merged commit 5fc7aa3 into main Feb 15, 2024
14 checks passed
@chris-rock chris-rock deleted the chris-rock/npm-resources branch February 15, 2024 18:53
@github-actions github-actions bot locked and limited conversation to collaborators Feb 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants