-
Notifications
You must be signed in to change notification settings - Fork 81
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
Add Requires.io badge to README #1541
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small thing...
README.rst
Outdated
@@ -66,6 +66,9 @@ Do this:: | |||
|
|||
.. |build-status-image| image:: https://secure.travis-ci.org/Cadasta/cadasta-platform.svg?branch=master | |||
:target: http://travis-ci.org/Cadasta/cadasta-platform?branch=master | |||
.. |req-status-image| image:: https://requires.io/github/Cadasta/cadasta-platform/requirements.svg?branch=master | |||
:target: https://requires.io/github/Cadasta/cadasta-platform/requirements.svg?branch=master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link points to the image, shouldn't it point to https://requires.io/github/Cadasta/cadasta-platform/requirements/?branch=master
@oliverroick Haha, that would be more helpful wouldn't it. Fixed! Thanks :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆
d2c2330
to
aee69b7
Compare
aee69b7
to
4133c96
Compare
* add Requires.io badge * Link the link
Proposed changes in this pull request
Adds Requires.io badge to README for package requirements status.
When should this PR be merged
Anytime
Risks
None; documentation only.
Follow-up actions
None
Checklist (for reviewing)
General
migration
label if a new migration is added.Functionality
Code
Tests
Security
Documentation