Skip to content

/video.js

Badges #3134

Closed
wants to merge 2 commits into from
+5 −1

Labels

confirmed patch

Milestone

No milestone

Assignee

No one assigned

2 participants

@gkatsev @mmcc
@gkatsev
Video.js member
gkatsev commented

Description

New shiny badges.

Specific Changes proposed

Move badges to new line.
Add coverage badge and nodeico badge.

Requirements Checklist

  • Reviewed by One Core Contributors
@gkatsev gkatsev Badges … 
Move badges to new line.
Add coverage badge and nodeico badge.
0adad64
@gkatsev gkatsev referenced this pull request
Closed

Update NPM to v5+ #3133

@gkatsev gkatsev put npm badge on a new line

All checks have passed

2 successful checks
Details continuous-integration/travis-ci/pr — The Travis CI build passed
Details coverage/coveralls — Coverage decreased (-0.7%) to 83.316%
65cf38b
@mmcc
Video.js member
mmcc commented

lgtm

@gkatsev gkatsev added confirmed and removed needs: LGTM labels
@gkatsev gkatsev added a commit that closed this pull request
@gkatsev gkatsev @gkatsev updated videojs badges in the README. closes #3134 d9a0a45
@gkatsev gkatsev closed this in d9a0a45
@gkatsev gkatsev deleted the gkatsev:badges branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.