-
Notifications
You must be signed in to change notification settings - Fork 10
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
Developer Documentation #189
Developer Documentation #189
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.
LGTM.
One comment is that please revert the changes that adding the line breaks, which are unnecessary.
Thank you!
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 for landing it. I might add some suggestions incrementally, but I can easily read the docus and I find them helpful for end users. Good job!
Thanks! I will improve it overtime. |
@jdneo We have android support PRs merged on both Gradle Build Server and Gradle for Java extension. There should be no blockers for this at the moment. |
04e7669
to
f9c0987
Compare
@Tanish-Ranjan Merged. Thanks for the contribution. |
This is the start for developer documentation covering my contributions to the repository. #185