npryce / adr-tools

Command-line tools for working with Architecture Decision Records
Other
4.63k stars 588 forks source link

Add option for having status values use shields.io #70

Closed eoghain closed 5 years ago

eoghain commented 6 years ago

Github users have been conditioned to look for shields.io tags as a quick way to understand the state of a repo. It'd be great if the Status section that is generated could replace the STATUS with a shields.io image instead of using just plain text.

Examples: Accepted
Rejected
Superceded

For clarity I'm thinking of a command line switch: adr --shields new URL Scheme adr --shields new -s 1 Updated URL Scheme

npryce commented 6 years ago

That would make it difficult to write scripts that work with ADR files. Maybe this could be done in the publishing pipeline, rather than maintained in the source

PeterMTaylor commented 6 years ago

To suggest from a Powershell point of view. This article shows a way that state of files being tested which updates the status badge directly to the readme.md itself. http://wragg.io/add-a-code-coverage-badge-to-your-powershell-deployment-pipeline/

eoghain commented 6 years ago

I don't see how it's any more difficult than plain strings. It's just a different regex. As long as the tool defines the the set of possible strings it'd be simple enough. Also I'm thinking of this as an optional choice that people can opt into using not a replacement for the plain strings.

npryce commented 5 years ago

I think that this is not a feature I can support in adr-tools, because:

eoghain commented 5 years ago

@npryce Thanks for taking a look. Everything you've said makes sense. Moving this into a visualizer is probably a good idea.