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 README.md #61

Merged
merged 1 commit into from
May 26, 2016
Merged

Update README.md #61

merged 1 commit into from
May 26, 2016

Conversation

dshil
Copy link

@dshil dshil commented May 26, 2016

Just add an installation instruction, think that repository will look better and people don't need to find how to get a source code.

Just add an installation instruction, think that repository will look better and people don't need to find how to get a source code.
@coveralls
Copy link

coveralls commented May 26, 2016

Coverage Status

Coverage remained the same at 96.352% when pulling 90ae818 on martinShil:master into 07c03a1 on uber-go:master.

@akshayjshah
Copy link
Contributor

Looks good. Thanks!

@akshayjshah akshayjshah merged commit 56e3723 into uber-go:master May 26, 2016
@akshayjshah
Copy link
Contributor

Hey, @martinShil - sorry, I completely forgot to ask you to sign our contributor license agreement before merging this. If you could, please sign the CLA. If you'd rather not sign the agreement, I can back out this change.

@dshil
Copy link
Author

dshil commented May 27, 2016

@akshayjshah, Of course I can. Maybe add some info about "contributor license agreement" in Readme. When new people will contribute, you don't need to ask them to sign your license.

@akshayjshah
Copy link
Contributor

Thanks! I wasn't expecting publicity or contributions quite so soon - added #64 to remind myself to add docs about this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants