Skip to content

Added additional badges to README.rst #40

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.

Sign up for GitHub

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

Jump to bottom
Merged
spatialbits merged 2 commits into from
Jan 2, 2018
Merged

Added additional badges to README.rst #40

spatialbits merged 2 commits into from
Jan 2, 2018

Conversation

@gregseb
Copy link
Member

@gregseb gregseb commented Dec 28, 2017 •
edited
Loading

  • Requires.io badge to show that our requirements are up to date
  • badge.fury.io to show our latest pypi release

@coveralls
Copy link

coveralls commented Dec 28, 2017

Coverage Status

Coverage remained the same at 93.976% when pulling 2cfa98c on new/require-and-version-badges into 43c400b on master.

1 similar comment
@coveralls
Copy link

coveralls commented Dec 28, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 93.976% when pulling 2cfa98c on new/require-and-version-badges into 43c400b on master.

@gregseb gregseb changed the title Added requires.io and badge.fury.io version badges to README.rst Added additional badges to README.rst Dec 29, 2017
@gregseb
Copy link
Member Author

gregseb commented Dec 29, 2017

This was originally a pull request specifically for the require.io and badge.fury.io badges, but I decided to try some additional badges and see how they look here.

@gregseb
Copy link
Member Author

gregseb commented Dec 29, 2017

Good collection of shields here: https://shields.io/

@gregseb
Copy link
Member Author

gregseb commented Dec 29, 2017

I wonder if its bad form to have two rows of badges? I don't want to go too badge crazy, but I kind of like having build badges and release badges separated.

@gregseb
Added two additional badges and altered one
70f83a5 
- Altered the badge.fury.io pypi version badge to use the shields.io equivalent instead
- Added a supported python versions badge from shields.io
- Added a license badge from shields.io
@coveralls
Copy link

coveralls commented Dec 29, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 93.976% when pulling 70f83a5 on new/require-and-version-badges into 43c400b on master.

@spatialbits
Copy link
Member

spatialbits commented Jan 2, 2018

Sure that looks ok. These are helpful badges. I agree we don't want too much clutter and this is probably enough badginess for now.

@spatialbits spatialbits merged commit 6dc9448 into master Jan 2, 2018
@gregseb gregseb deleted the new/require-and-version-badges branch November 2, 2018 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gregseb @coveralls @spatialbits