[clazy] [Bug 474657] clazy is showing no-module-include issues as "errors" not "warnings"

[Jazeix Johnny]

https://bugs.kde.org/show_bug.cgi?id=474657

Jazeix Johnny <jazeix at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REPORTED                    |ASSIGNED
     Ever confirmed|0                           |1

--- Comment #5 from Jazeix Johnny <jazeix at gmail.com> ---
(In reply to Jazeix Johnny from comment #4)
> I think the issue here is that you include directly the module
> QtPrintSupport which, as other Qt modules, include all the modules and not
> the relevant classes
> (https://codebrowser.dev/qt5/qtbase/include/QtPrintSupport/QtPrintSupport.
> html).
> 
> Wouldn't it be better to add it in the list of warnings to tell the user to
> directly use the relevant classes of QtPrintSupport
> (https://invent.kde.org/sdk/clazy/-/commit/
> b205b52c0efbee044fd9377e077cb73e8772c8f8#dd685b92c5cf09d7fb134070ca5b0febc6e8
> b8e3_0_40)?

I've just checked and we already ignore when it's included from a Qt module,
the issue is that I missed the QtPrintSupport module. I did it manually, I'm
not sure if there is an automatic way to list of the Qt modules to ensure I
didn't miss more

-- 
You are receiving this mail because:
You are the assignee for the bug.