doc: update badges in README.md and doc/index.md #71
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
belochub
approved these changes
Feb 13, 2017
|
@tshemsedinov what's the status of this and metarhia/Metarhia#3? |
There was a problem hiding this comment.
I'd like to propose:
- BitHound score:
[](https://www.bithound.io/github/metarhia/JSTP)
- NPM version using badge.fury.io
[](https://badge.fury.io/js/metarhia-jstp)
|
@tshemsedinov +1 for the BitHound score. What about the second proposal, |
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3
|
@tshemsedinov I updated the commit to address the suggestions. Does this LGTY? |
tshemsedinov
approved these changes
Feb 19, 2017
aqrln
added a commit
that referenced
this pull request
Feb 19, 2017
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3 PR-URL: #71
|
Landed in 28fcd10. |
aqrln
added a commit
that referenced
this pull request
Feb 20, 2017
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3 PR-URL: #71
Merged
aqrln
added a commit
that referenced
this pull request
Feb 20, 2017
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3 PR-URL: #71
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3 PR-URL: #71
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
Travis CI and bitHound badges in `README.md` are now included directly. NPM downloads stats badges are left with <shields.io> since there are no official badges from NPM itself and these were generated by <shields.io>, not included from an external service. Yet their style is changed to match the first two ones. NPM version badge is now included from <badge.fury.io>. Badges in `doc/index.md` are now synchronized with badges in `README.md` since they weren't updated last time when they were changed in `README.md`. Refs: metarhia/Metarhia#3 PR-URL: #71
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Travis CI and bitHound badges in
README.mdare now included directly.NPM badges are left with <shields.io> since there are no official badges
from NPM itself and these were generated by <shields.io>, not included
from an external service. Yet their style is changed to match the first
two ones. Badges in
doc/index.mdare now synchronized with badges inREADME.mdsince they weren't updated last time when they were changedin
README.md.Refs: metarhia/Metarhia#3
Pro: more stable badge loading.
Con: badges aren't flat anymore.