Skip to content

Commit

Permalink
chores: removed npm-auditor package and the corresponding gulp task (#…
Browse files Browse the repository at this point in the history
…8143)

<!--
Thank you for your contribution.

Before making a PR, please read our contributing guidelines at

https://github.com/DevExpress/testcafe/blob/master/CONTRIBUTING.md#code-contribution

We recommend creating a *draft* PR, so that you can mark it as 'ready
for review' when you are done.
-->

## Purpose
Removed npm-auditor package and the corresponding gulp task in order to
get rid of ip package vulnerability.

## Approach
Deleted gulp task and npm uninstall npm-auditor package

## References

https://github.com/orgs/DevExpress/projects/11/views/1?pane=issue&itemId=52847042

## Pre-Merge TODO
- [ ] Write tests for your proposed changes
- [ ] Make sure that existing tests do not fail

Co-authored-by: Bayheck <adil.rakhaliyev@devexpress.com>
  • Loading branch information
Bayheck and Bayheck committed Feb 20, 2024
1 parent 15cfdca commit 23c7811
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 526 deletions.
12 changes: 0 additions & 12 deletions Gulpfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ const globby = require('globby');
const minimist = require('minimist');
const functionalTestConfig = require('./test/functional/config');
const childProcess = require('child_process');
const npmAuditor = require('npm-auditor');
const checkLicenses = require('./test/dependency-licenses-checker');
const packageInfo = require('./package');
const ensureDockerEnvironment = require('./gulp/docker/ensure-docker-environment');
Expand Down Expand Up @@ -66,17 +65,6 @@ process.env.PATH = NODE_MODULE_BINS + path.delimiter + process.env.PATH + path.d

process.env.DEV_MODE = ('dev' in ARGS).toString();

gulp.task('audit', () => {
return npmAuditor()
.then(result => {
process.stdout.write(result.report);
process.stdout.write('\n');

if (result.exitCode !== 0)
throw new Error('Security audit failed');
});
});

gulp.task('clean', () => {
return del('lib');
});
Expand Down
Loading

0 comments on commit 23c7811

Please sign in to comment.