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
Add the ability to transform a license url with a regular expression/replacement pair. #167
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k-wall could you please rebase on top of current master? I have recently merged a couple of older PRs and added style checks.
Rebase done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please make mvn checkstyle:check
pass?
Rebase done and check style problems resolved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k-wall thanks, see the comments above.
Please reword the commit to start with Fix #154
src/main/java/org/codehaus/mojo/license/LicenseUrlReplacement.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/LicenseUrlReplacement.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/LicenseUrlReplacement.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java
Outdated
Show resolved
Hide resolved
@k-wall any chance that you find time to address the review comments above? I'd like to merge and release the plugin. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k-wall I see a new commit but I do not see any of the review comments resolved.
That was my rebase before starting work. I am currently addressing the
review comments. I should have a patch up with changes shortly.
…On Wed, 9 Jan 2019 at 10:14, Peter Palaga ***@***.***> wrote:
***@***.**** requested changes on this pull request.
@k-wall <https://github.com/k-wall> I see a new commit but I do not see
any of the review comments resolved.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#167 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ARlgOh6DqinwQXba2uZiF97yx7J-Cbseks5vBcD5gaJpZM4Zgk13>
.
|
Thanks for the comments, I believe all are now resolved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally OK, thanks. I have added a few minor comments.
src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/license/AbstractDownloadLicensesMojo.java
Outdated
Show resolved
Hide resolved
@k-wall one more detail: Please squash all your commits into one, and use the follwing commit message: |
…gular expression/replacement pair
@ppalaga I've addressed your comments, reworded the commit message to refer to the #196, and rebased/squashed. I also attach a diff showing the code changes made to address your latest review to help you review efficiently. 0001-Changes-in-response-to-ppalaga-review-2019-01-10.patch.zip |
@k-wall thanks a lot! |
Allows a project to, say, workaround a project's duff or stale license metadata.
Fixes #154