Uploaded image for project: 'Qt Tools for Android'
  1. Qt Tools for Android
  2. QTTA-158

QtGP Add Types for QtGP Test classes

    XMLWordPrintable

Details

    • Android
    • 8ef11ddba (dev)

    Description

      Why?

      Cause

      • To get rid of unnecessary warnings in tests.
        • There are many places where we are using an extension or build task and it shows a yellow highlight or a gray method name even though it exists.
      • To make the code more easily readable, thanks to better Groovy syntax highlighting.

      Customer

      •  

      Cruft

      •  

      What?

      Definition

      • Change functions from return type "def" to typed: void, Project, QtBuildTask...
      • For example QtPluginExtension has qtPath and in IntellJ it shows: "No candidates found for method call pluginExtension. qtPath.".

      Demarcation

      • Does not include code cleaning, after changes are done ticket: QTTA-181

      Dependencies

      •  

      Attachments

        1. AfterCanditateForFunctionFound.png
          30 kB
          Tero Koponen
        2. AfterQtKitDirNoWarning.png
          17 kB
          Tero Koponen
        3. BeforeNoCanditateFound.png
          33 kB
          Tero Koponen
        4. BeforeQtKitDirWarning.png
          16 kB
          Tero Koponen
        5. image-2024-09-12-09-24-24-034.png
          16 kB
          Tero Koponen
        6. image-2024-09-12-09-25-05-160.png
          33 kB
          Tero Koponen
        7. Screenshot 2024-10-02 093409.png
          17 kB
          Tero Koponen
        8. Screenshot 2024-10-02 093754.png
          30 kB
          Tero Koponen

        Issue Links

          For Gerrit Dashboard: QTTA-158
          # Subject Branch Project Status CR V

          Activity

            People

              teko Tero Koponen
              teko Tero Koponen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes