README: update badges - Githubissues

sous-chefs / openvpn

Development repository for the openvpn cookbook

https://supermarket.chef.io/cookbooks/openvpn

Apache License 2.0

98 stars 160 forks source link

README: update badges #141

Closed josephholsten closed 5 years ago

josephholsten commented 5 years ago

Description

change build status badge from travis to circleci
remove license badge

Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

flaccid commented 5 years ago

Possible to add license badge back?

damacus commented 5 years ago

I don't really understand the use of the license badge. Seeing as it's at the bottom of the readme. How's it help you?

flaccid commented 5 years ago

User sees straight up at the top visually what the license is :)

damacus commented 5 years ago

Honestly just seems like more cruft to maintain in this case. :-1:

flaccid commented 5 years ago

Convenience at its worst :)

josephholsten commented 5 years ago

I convinced myself that it wasn't valuable because github shows the license above the fold on the github repo page.

I don't think other readers of the README need to know unless they are going to contribute. And the only other situation where a contributor will see this will be while editing the readme in a text editor, and a literal [![License](https://img.shields.io/badge/license-Apache_2-blue.svg)](https://www.apache.org/licenses/LICENSE-2.0) isn't exactly the most readable way to describe the license in that context.

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.