Uploaded image for project: 'Build'
  1. Build
  2. BUILD-285

Further define the effort for clean dependency analysis reports

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • POMs 35
    • None
    • None
    • None
    • Basel 123, Basel 124, Basel 125, Basel 126
    • 13

      Following the initial POC done in DEV-676, here are some points that need to be adressed before we implement such a health check:

      1. javax.servlet-api is used in a lot of modules, often declared as unused, but tests fail in some cases if it is not present (~RepositoryTestCase). Investigate if that could be fixed, allowing us to remove the dependency everywhere. those will be ignored thanks to the wildcard rule, they're not really false positive, though - but an investigation into the topic would be a task on its own
      2. javax.jcr and javax.inject are used almost everywhere but not necessarily explicitely declared. Do we want to have them as part of every POM? Near the bottom? yes, they'll be added. Order TBD exactly in a later step
      3. Investigate if it is possible with a reasonable effort to have yellow rather than red builds for failing analysis reports. in the implementation phase
      4. In the backup module, does h2 need to be scoped appropriately (rather than excluded) to no longer be an error? not a scope issue, no
      5. Reactor vs. parent POM: which responsibilities does each have? no longer relevant AFAICS
      6. Investigate why, in site/site-app, mvn says mockito-core and magnolia-site are superfluous dependencies, which they are not at all. Same with mockito-core in templating-samples. false positives that will be ignored once PR is in place
      7. In UI, org.vaadin.addon:easyuploads:jar:8.0.0:compile is a false positive in submodules that don't even use it. Investigate. false positive that will be ignored once PR is in place
      8. In groovy/magnolia-module-groovy, com.google.gwt:gwt-elemental:jar:2.8.1:provided is both a problem if you add or remove it. false positive that will be ignored once PR is in place
      9. com.google.code.findbugs:annotations used in resources and main wasn't added to the BOM but individually to those two projects - apchelintcev thought we might use other impls. of nullability annotations. Also the project seems dead. see BUILD-291
      10. usages of the icons project are not caught by Maven. What to do? false positive that will be ignored once PR is in place
      11. how to skip the webapps? this could be done in a Jenkins pipeline file if that's what we end up doing. Let's see in the impl phase
      12. investigate comment by Michael below. equivalent to what we're doing

        Acceptance criteria

              mmichel Maxime Michel
              mmichel Maxime Michel
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Task DoR