Skip to content

Conversation

@viddo
Copy link

@viddo viddo commented Dec 5, 2025

Summary

Fixes #291 by adding support for loading SuppressionsService from ESLint that is currently used from CLI.

What kind of change does this PR introduce?

feature/bugfix

Did you add tests for your changes?

Does this PR introduce a breaking change?

No, it only amends/fix the suppressions feature.

If relevant, what needs to be documented once your changes are merged or what have you already documented?

New option suppressionsLocation to indicate where the suppressions file is located (if not on default location).

@linux-foundation-easycla
Copy link

CLA Not Signed

@viddo viddo changed the title feat: Fix errors despite bulk suppressions feat: support bulk suppressions Dec 5, 2025
Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Can you accept CLA? Thank you

*/
function getSuppressionsService() {
// ESLint doesn't export SuppressionsService in package.json exports,
// so we need to resolve the path directly
Copy link
Member

Choose a reason for hiding this comment

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

We need to open an issue in eslint repo, ideally they should export or provide API for such things

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.

Errors despite bulk suppressions

2 participants