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

SDK-2568: Evaluator configuration file docs #2179

Merged
merged 9 commits into from
Sep 20, 2023

Conversation

sergeyspryker
Copy link
Contributor

PR Description

https://spryker.atlassian.net/browse/SDK-2568

TBD

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@sergeyspryker sergeyspryker added the new PR contains a new document label Sep 14, 2023
@DmytroKlymanSpryker DmytroKlymanSpryker added the TW review needed PR needs Technical writer's review label Sep 18, 2023
@DmytroKlymanSpryker DmytroKlymanSpryker changed the title SDK-2568 evaluator configuration file docs SDK-2568: Evaluator configuration file docs Sep 19, 2023
## Ignoring checkers errors

To ignore errors need to specify regular expressions in `ignoreErrors` section in configuration file. Each error message that matches these regular expressions will be ignored by evaluator.
You can specify rules both for some specific checkers or globally for all of them.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Previously, you've mentioned that in this config, we can add errors to ignore.
Here, you say that we can add *checkers to ignore.

So what is it exactly that we can ignore?

@andriitserkovnyi andriitserkovnyi merged commit 1355f40 into master Sep 20, 2023
@andriitserkovnyi andriitserkovnyi deleted the feature/sdk-2568-regexp-error-filter branch September 20, 2023 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document TW review needed PR needs Technical writer's review
Development

Successfully merging this pull request may close these issues.

3 participants