Suppress superfluous multi-rule per checker warnings #27
Conversation
This warning by baselib is intrusive and currently makes no sense for OSI-rules based checks due to the high number of rules to be checked. Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
TimmRuppert
left a comment
TimmRuppert
left a comment
There was a problem hiding this comment.
I agree. Did you not already raise an issue or comment somewhere in qc-framework / qc-baselib in order to express our wish for change upstream? I seem to cannot find it.
|
There is now a PR that does this, asam-ev/qc-baselib-py#59, but this will likely not make it into this release, and the workaround here is stable even if we fix this in upstream. |
thomassedlmayer
left a comment
thomassedlmayer
left a comment
There was a problem hiding this comment.
Log suppression works as expected for me.
asadekasam
added
the
ReadyForCCBReview
|
This PR can for now be postponed, as a fix in qc-baselib-py will be part of the 1.1.0 release of the framework (release decision coming up on 2025-11-17), see asam-ev/qc-baselib-py#59. Will close once release has been decided. |
pmai
removed
the
ReadyForCCBReview
|
P_2025_02_008 project meeting: This shall be closed as it was resolved with asam-ev/qc-baselib-py#59 and is now part of the QC 1.1.0 release |
This warning by baselib is intrusive and currently makes no sense for OSI-rules based checks due to the high number of rules to be checked.