-
Notifications
You must be signed in to change notification settings - Fork 21
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 commit hash discovery #107
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.
Hi!
First of all, thank you for opening your first pull-request. Elementary, we appreciate all submissions that helps us continue improving this plugin.
Please take a look to our contributing guidelines and make sure all requirements are fulfilled.
As this is a community project we can't commit to official due dates for reviews and developing, but we're definitely committed to delivering services, integrations and plugins of top quality.
So please be patient, we will review your pull-request and get back to you as soon as we can!
Regards,
Sherlock Trunks 🐘
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.
LGTM
src/main/java/org/sonatype/gradle/plugins/scan/nexus/iq/scan/NexusIqPluginScanExtension.java
Outdated
Show resolved
Hide resolved
src/main/java/org/sonatype/gradle/plugins/scan/nexus/iq/scan/NexusIqPluginScanExtension.java
Show resolved
Hide 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.
+1
Thanks for your contribution @eduard-tita 👍 |
This pull request makes the following changes:
scanFolderPath
field inNexusIqPluginScanExtension
(the current value will never work).scanFolderPath
field in order to properly collect the commit hash during scans.It relates to the following issue #s:
cc @bhamail / @DarthHater / @guillermo-varela / @shaikhu