Skip to content

added --check-library to selfchecks - #3082

Draft
firewave wants to merge 2 commits into
cppcheck-opensource:mainfrom
firewave:self-check-library
Draft

added --check-library to selfchecks#3082
firewave wants to merge 2 commits into
cppcheck-opensource:mainfrom
firewave:self-check-library

Conversation

@firewavefirewave changed the title CI-unixish.yml: added --check-library to self-check and enabled information messagesCI-unixish.yml: added --check-library to self-checkMar 31, 2021
@firewave

This comment was marked as resolved.

@firewavefirewave changed the title CI-unixish.yml: added --check-library to self-checkadded --check-library to self-checkMar 31, 2021
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

This shows that --check-library has way too many false positives. I will try to file tickets for some of the cases.

@firewave

This comment was marked as resolved.

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

There are way too many messages being reported which will (and possibly can) never be addressed completely. I wonder if we should make --check-library more granular so it might be possible to least tackle and enable a subset of the reported stuff.

@firewave
firewaveforce-pushed the self-check-library branch from 9ba6fae to 0e6c3bbCompareJuly 11, 2022 10:50
@firewavefirewave changed the title added --check-library to self-checkadded --check-library to self-checkJul 11, 2022
@firewave
firewaveforce-pushed the self-check-library branch 3 times, most recently from 4570257 to 1da497dCompareJuly 13, 2022 19:12
@firewave
firewaveforce-pushed the self-check-library branch from 1da497d to 4efbd52CompareJuly 19, 2022 19:09
@firewave
firewaveforce-pushed the self-check-library branch 2 times, most recently from 80194bf to 70717feCompareJanuary 26, 2023 21:42
@firewave
firewaveforce-pushed the self-check-library branch 2 times, most recently from 7ea2b4a to 8167801CompareFebruary 8, 2023 11:33
@firewave
firewaveforce-pushed the self-check-library branch 2 times, most recently from 48fe178 to 02e8e0cCompareFebruary 24, 2023 23:32
@firewave
firewaveforce-pushed the self-check-library branch from 4c22214 to d1b30f5CompareJuly 26, 2023 17:52
@firewave
firewaveforce-pushed the self-check-library branch 2 times, most recently from efcf9fd to c11aeceCompareJanuary 6, 2024 15:17
@firewave
firewaveforce-pushed the self-check-library branch from c11aece to eb5ad8aCompareMarch 3, 2024 16:59
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

@orbitcowboy Maybe you could also look at the warnings reported for our own code? Thanks.

Also mind the linked tickets which might require some updating after all your additions.

@firewave
firewaveforce-pushed the self-check-library branch from eb5ad8a to 119b368CompareMay 1, 2024 18:18
@firewave
firewaveforce-pushed the self-check-library branch 2 times, most recently from b263e58 to dd2539fCompareSeptember 29, 2024 22:56
@firewavefirewave changed the title added --check-library to self-checkadded --check-library to selfchecksFeb 18, 2026
@sonarqubecloud

Copy link
Copy Markdown

Sign up for freeto 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.

1 participant

@firewave