Skip to content
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

update badges and add tags to metadata #412

Merged
merged 1 commit into from
Oct 26, 2017
Merged

update badges and add tags to metadata #412

merged 1 commit into from
Oct 26, 2017

Conversation

wyardley
Copy link
Contributor

Update badges in README based on other modules.

This might also help until the issues discussed in #32 are addressed?

[![Downloads](https://img.shields.io/puppetforge/dt/puppet/corosync.svg)](https://forge.puppet.com/puppet/corosync)
[![Build Status](https://travis-ci.org/voxpupuli/puppet-corosync.png?branch=master)](https://travis-ci.org/voxpupuli/puppet-corosync)
[![Endorsement](https://img.shields.io/puppetforge/e/puppet/corosync.svg)](https://forge.puppet.com/approved)
[![CII Best Practices](https://bestpractices.coreinfrastructure.org/projects/533/badge)](https://bestpractices.coreinfrastructure.org/projects/533)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you not like this one?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can add it back, I didn't see it in our other ones, and thought we were trying to keep a standard set? Re-added.
Also the scores one doesn't seem to be working AFAICT - I get https://img.shields.io/puppetforge/f/puppet/corosync.svg 'score unknown' for this despite the forge having a score (same for mongodb)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes the CII badge is proving the quality of this repo. We should instead add it to other repositories.

@wyardley
Copy link
Contributor Author

@alexjfisher tests are passing now. the CII badge is added back.

Copy link
Member

@dhollinger dhollinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but I'd like @alexjfisher or @roidelapluie to review as well

@dhollinger dhollinger merged commit a662fde into voxpupuli:master Oct 26, 2017
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