Skip to content

Exclude eslint as a SAST analyzer

Thomas Woodham requested to merge 12296-disable-eslint into master

What does this MR do?

The eslint analyzer that's part of SAST is quite noisy and has a pretty high false positive rate. Also, it's coverage is now duplicated by the semgrep analyzer that also provides coverage for JavaScript. This MR removes eslint from the GitLab build configuration.

Related issues

Check-list

Pre-merge

Consider the effect of the changes in this merge request on the following:

If new jobs are added:

This will help keep track of expected cost increases to the GitLab project average pipeline cost per merge request RPI

Post-merge

Merge request reports

Loading