8000 chore: drop jazzer and rxjs devDependencies by karfau · Pull Request #845 · xmldom/xmldom · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: drop jazzer and rxjs devDependencies #845

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
Merged
merged 3 commits into from
Feb 25, 2025
Merged

chore: drop jazzer and rxjs devDependencies #845

merged 3 commits into from
Feb 25, 2025

Conversation

karfau
Copy link
Member
@karfau karfau commented Feb 25, 2025 •
edited
Loading
  • jazzer has been deprecated and has tons of deprecated dependencies, and I'm no longer able to install it on my machine, without adding prebuild as a devDependency which adds some more deprecated dependencies to the project...
npm error code 127
npm error path /home/karfau/dev/xmldom/node_modules/@jazzer.js/fuzzer
npm error command failed
npm error command sh -c prebuild-install --runtime napi || npm run prebuild
npm error > @jazzer.js/fuzzer@2.1.0 prebuild
npm error > prebuild --runtime napi --backend cmake-js --all --strip --verbose
npm error prebuild-install warn This package does not support N-API version undefined
npm error prebuild-install warn install No prebuilt binaries found (target=undefined runtime=napi arch=x64 libc= platform=linux)
npm error sh: line 1: prebuild: command not found

@karfau
chore: drop jazzer and rxjs devDependencies
3be5945 
- jazzer has been deprecated and has tons of deprecated dependencies
- rxjs is only used as a transient dependency
@socket-security Socket Security
Copy link
socket-security bot commented Feb 25, 2025 •
edited
Loading

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/@jazzer.js/core@2.1.0, npm/@jazzer.js/jest-runner@2.1.0

View full report↗︎

@codecov Codecov
Copy link
codecov bot commented Feb 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.12%. Comparing base (7fc02cb) to head (c04267e).
Report is 1 commits behind head on master.

Additional details and impacted files 
@@           Coverage Diff           @@
##           master     #845   +/-   ##
=======================================
  Coverage   95.12%   95.12%           
=======================================
  Files           8        8           
  Lines        2195     2195           
  Branches      577      577           
=======================================
  Hits         2088     2088           
  Misses        107      107

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@karfau karfau merged commit 00a686f into master Feb 25, 2025
38 checks passed
@karfau karfau deleted the drop-jazzer branch February 25, 2025 15:35
@karfau karfau mentioned this pull request Feb 27, 2025
Merged
DavidKorczynski pushed a commit to google/oss-fuzz that referenced this pull request Mar 1, 2025
@karfau
xmldom: install @jazzer.js/core as part of build.sh (#13107)
1ea07fc 
it was [dropped from `devDependencies` in
xmldom](xmldom/xmldom#845), since it can no
longer be installed reliably on all machines.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@karfau
0