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.

Sign up for GitHub

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

Jump to bottom

Add licenseMergesUrl also in report goals #253

Closed
nfalco79 opened this issue Feb 12, 2019 · 0 comments · Fixed by #290
Closed

Add licenseMergesUrl also in report goals #253

nfalco79 opened this issue Feb 12, 2019 · 0 comments · Fixed by #290
Labels
enhancement third-party-report
Milestone
1.18

Comments

@nfalco79
Copy link
Contributor

nfalco79 commented Feb 12, 2019 •
edited
Loading

Existing report goals:

  • third-party-report
  • aggregate-third-party-report

does not support licenseMergesUrl like their conterpart:

  • add-third-party
  • aggregate-add-third-party
@elballa elballa mentioned this issue Feb 15, 2019
Merged
ppalaga added a commit that referenced this issue Feb 19, 2019
@ppalaga
Merge pull request #262 from elballa/feature/253
9ae1c6b 
Fix #253 Add licenseMergesUrl also in report goals
@ppalaga ppalaga added this to the 1.18 milestone Feb 20, 2019
elballa pushed a commit to elballa/license-maven-plugin that referenced this issue Feb 25, 2019
@balessandr
Fix mojohaus#253 Add licenseMergesUrl also in report goals
8f3b993 
There is still something missing in this fix mojohaus#253. It should be included in next release because licenseMergeUrl in not reliable if there is more then one raw in content of file linked by licenseMergeUrl
@elballa elballa mentioned this issue Feb 25, 2019
Merged
elballa pushed a commit to elballa/license-maven-plugin that referenced this issue Feb 25, 2019
@balessandr
Fix mojohaus#253 Add licenseMergesUrl also in report goals
b7439a9 
There is still something missing in this fix mojohaus#253. It should be included in next release because licenseMergeUrl in not reliable if there is more then one raw in content of file linked by licenseMergeUrl
elballa pushed a commit to elballa/license-maven-plugin that referenced this issue Feb 25, 2019
@balessandr
Fix mojohaus#253 Add licenseMergesUrl also in report goals
652cb45 
Failure execution
elballa pushed a commit to elballa/license-maven-plugin that referenced this issue Feb 25, 2019
@balessandr
Fix mojohaus#253 Add licenseMergesUrl also in report goals
0192c5b 
There is still something missing in this fix mojohaus#253. It should be included in next release because licenseMergeUrl in not reliable if there is more then one raw in content of file linked by licenseMergeUrl
ppalaga added a commit that referenced this issue Feb 25, 2019
@ppalaga
Merge pull request #290 from elballa/feature/253
7d3a633 
Fix #253 Add licenseMergesUrl also in report goals
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement third-party-report
Projects
None yet
Milestone
1.18
Development

Successfully merging a pull request may close this issue.

Fix #253 Add licenseMergesUrl also in report goals elballa/license-maven-plugin
2 participants
@ppalaga @nfalco79