-
Notifications
You must be signed in to change notification settings - Fork 59
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
Publish artifacts to maven local #92
Publish artifacts to maven local #92
Conversation
Signed-off-by: Owais Kazi <[email protected]>
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.
Take care of good comments by @dbwiddis , otherwise looks good to me.
Care to update our developer guide?
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.
Other than the comments, also looks good to me!
Signed-off-by: Owais Kazi <[email protected]>
04e8eef
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 except you resolved the comment about updating settings.gradle
but didn't change it in this PR. Either change it here or wait until #96 is merged before merging this.
Signed-off-by: Owais Kazi <[email protected]>
Pushed the change 🙂 |
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
* Publish artifacts to maven local Signed-off-by: Owais Kazi <[email protected]> * PR comments and updated the dev guide Signed-off-by: Owais Kazi <[email protected]> * Renamed project in settings.gradle Signed-off-by: Owais Kazi <[email protected]> Signed-off-by: Owais Kazi <[email protected]>
Signed-off-by: Owais Kazi [email protected]
Description
Publish artifacts to Maven and use SDK as a library. This will help us to just import the code from SDK rather than having the whole SDK package in AD extension.
Issues Resolved
Closes #98
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.