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.

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

Update the project setup guide / Update dependencies #109

Merged
merged 1 commit into from
Feb 15, 2016

Conversation

trustin
Copy link
Member

@trustin trustin commented Feb 15, 2016

  • Split the setup guide to two sub-pages; one for Maven and the other for Gradle
  • Use text substitutions for most dependency versions in the setup guide
  • Improve the access to the pom.xml in the Sphinx conf.py
  • Bump the micro versions of logback-classic, slf4j and jackson

@trustin trustin added this to the 0.9.0.Final milestone Feb 15, 2016
@trustin trustin force-pushed the update_site_snippets branch 3 times, most recently from 57c53ef to 5e8cb97 Compare February 15, 2016 05:56
@@ -52,6 +52,22 @@ like the following:
}
});

You can also use builder pattern for richer client construction:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the builder pattern.

Also not sure what you mean by richer, maybe we can just leave the adjective out.

- Split the setup guide to two sub-pages; one for Maven and the other
  for Gradle
- Use text substitutions for most dependency versions in the setup guide
- Improve the access to the pom.xml in the Sphinx conf.py
- Bump the micro versions of logback-classic, slf4j and jackson
@anuraaga
Copy link
Collaborator

LGTM

trustin added a commit that referenced this pull request Feb 15, 2016
Update the project setup guide / Update dependencies
@trustin trustin merged commit d5a1139 into line:master Feb 15, 2016
@trustin trustin deleted the update_site_snippets branch February 15, 2016 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants