Skip to content

Readme updates#26

Merged
dh-- merged 4 commits intomasterfrom
readme-updates
Sep 24, 2014
Merged

Readme updates#26
dh-- merged 4 commits intomasterfrom
readme-updates

Conversation

@dh--
Copy link
Copy Markdown
Contributor

@dh-- dh-- commented Sep 23, 2014

Updating the readme with a few formatting and superficial changes. Also, including a comment about support (thanks Mark!) and additional choice links to developers.google.com.

A bunch of updates to the read me file. Most of it is surface level
tweaking. I’ve also included instructions on how to get an API key, and
a few additional links to choice pages on developers.google.com.
README.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd like to say something like "we reserve the right to make backwards-incompatible changes while the version is v0.*". i.e. in line with semantic versioning.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@broady What do you think?

I'm worried calling it out weakens the comment of "we want you to build real production applications."
Will defer to the majority.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@broady
Copy link
Copy Markdown
Contributor

broady commented Sep 23, 2014

LGTM after Mark's comments are resolved 👍

 - clarifying java version
 - lower case ‘keys’
 - small code change when setting your key
dh-- added a commit that referenced this pull request Sep 24, 2014
Updates to the ReadMe. Cleaning up the original documentation, and clarifying some things around support/contribution.
@dh-- dh-- merged commit 4f64746 into master Sep 24, 2014
@broady broady deleted the readme-updates branch October 4, 2014 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants