Skip to content

Add a license badge to README.md #556

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

Merged
merged 1 commit into from
Jun 3, 2015
Merged

Add a license badge to README.md #556

merged 1 commit into from
Jun 3, 2015

Conversation

ssokolow
Copy link

@ssokolow ssokolow commented Jun 3, 2015

Also, move the crates.io badge beyond the license badge to optimize the badge
row's utility as a status dashboard as discussed in #525.

@GitCop
Copy link

GitCop commented Jun 3, 2015

Thanks for contributing! Unfortunately, I'm here to tell you there were the following style issues with your Pull Request:

  • Commit: 81e4965
    • Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md


This message was auto-generated by https://gitcop.com

@ssokolow
Copy link
Author

ssokolow commented Jun 3, 2015

Whoops. One sec while I force-push a commit fix.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.78% when pulling 81e4965 on ssokolow:patch-1 into 486a219 on hyperium:master.

@ssokolow
Copy link
Author

ssokolow commented Jun 3, 2015

There. Sorry about that. My sleep cycle is somewhat disrupted and I forgot to consider how it subtly affects my memory and judgement.

EDIT: ...and I just realized that my writing instincts overrode the "don't capitalize first letter" that I'd just read. One sec.

1. Add a shields.io license badge to allow quick, effective identification
   of the license without requiring the user to scroll to the bottom of the
   page.
2. Remove the single-word license section at the bottom of the page as it
   is now superfluous and looks unpolished.
3. Move the crates.io badge beyond the license badge so that its orange
   colour won't knock the viewer's brain off fast-path evaluation when
   using the badge row as a status dashboard (as explained in #525).

Fixes #525
@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.78% when pulling 19d2530 on ssokolow:patch-1 into 486a219 on hyperium:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.78% when pulling 19d2530 on ssokolow:patch-1 into 486a219 on hyperium:master.

seanmonstar added a commit that referenced this pull request Jun 3, 2015
Add a license badge to README.md
@seanmonstar seanmonstar merged commit e2611b4 into hyperium:master Jun 3, 2015
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.

4 participants