Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #413 Use 'GAV' for result keys in LicensedArtifactResolver #414

Merged
merged 3 commits into from
Jul 2, 2023
Merged

Fix #413 Use 'GAV' for result keys in LicensedArtifactResolver #414

merged 3 commits into from
Jul 2, 2023

Commits on Sep 23, 2021

  1. Use 'GAV' for result keys in LicensedArtifactResolver

    Fixes #413
    
    In the result mapping we need to use "GAV" to be compatible with
    the other classes in the plugin. While the exclude/includeArtifacts
    should still use the Artifact#getId, since the getDependencyTrail
    also uses that format, which is considered when we exclude all the
    transitive deps of excluded artifacts.
    
    Note that the added it is based on the already existing
    download-licenses-include-exclude-types, but with a small change
    in the settings to use excludeTransitiveDependencies instead.
    Attila Puskas committed Sep 23, 2021
    Configuration menu
    Copy the full SHA
    120f372 View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2023

  1. fix CRLF

    slawekjaranowski committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    cc59d7a View commit details
    Browse the repository at this point in the history
  2. Merge remote-tracking branch 'origin/master' into pr-414

    # Conflicts:
    #	src/main/java/org/codehaus/mojo/license/download/LicensedArtifactResolver.java
    slawekjaranowski committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    f3fa76a View commit details
    Browse the repository at this point in the history