Skip to content

Add warning about filtering#240

Open
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:documentation-how-to-skip-analysis
Open

Add warning about filtering#240
furtib wants to merge 3 commits into
Ericsson:mainfrom
furtib:documentation-how-to-skip-analysis

Conversation

@furtib

@furtib furtib commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Why:
Users are confused about why --test_tag_filters=-codechecker does not skip expensive analysis jobs.

What:

  • Added a warning explaining how to correctly filter codechecker tests.

Addresses:
#238

@furtib furtib requested a review from Szelethus June 11, 2026 16:04
@furtib furtib self-assigned this Jun 11, 2026
@furtib furtib added the documentation Improvements or additions to documentation label Jun 11, 2026
Comment thread README.md Outdated
@furtib furtib requested a review from Szelethus June 12, 2026 08:50
Comment thread README.md Outdated
```

> [!WARNING]
> Filtering codecheckerr_tests with only `--test_tag_filters=-codechecker` is not enough.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
> Filtering codecheckerr_tests with only `--test_tag_filters=-codechecker` is not enough.
> Filtering codechecker_tests with only `--test_tag_filters=-codechecker` is not enough.

Okay, this is great info, but is this expected or desired behaviour?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not desired, but users should know of it until we write a fix for it.

@furtib furtib requested a review from Szelethus June 12, 2026 11:27
furtib and others added 3 commits June 12, 2026 13:44
@furtib furtib force-pushed the documentation-how-to-skip-analysis branch from d1223d8 to 99f2e15 Compare June 12, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants