llnode icon indicating copy to clipboard operation
llnode copied to clipboard

doc: use official CI badge

Open gengjiawen opened this issue 4 years ago • 2 comments

gengjiawen avatar Mar 05 '20 08:03 gengjiawen

Humm, will check tomorrow, but the official one seems to be for the last status regardless if the build was on a PR or on master, whereas the shields badge will get the last result only from master? Otherwise lgtm

The shields version is broken, not sure why. As the icon page suggest the badge is for the default branch. You can lock to master branch too.

gengjiawen avatar Mar 05 '20 08:03 gengjiawen

Oh, nice catch. There was a typo on the Shields URL (push instead of on%20push%20or%20pull_request). Do you mind updating the URL to keep using shield (so that all three badges look the same)? The correct url should be https://img.shields.io/github/workflow/status/nodejs/llnode/on%20push%20or%20pull_request/master?style=flat-square

mmarchini avatar Mar 06 '20 23:03 mmarchini

@gengjiawen Apologies for having this open for so long but now we have landed the work to support 16/18 I am looking to clean up the outstanding PRs. Do you want to sync your PR and apply the final change that Mary requested? i.e. apply the flat-style property https://img.shields.io/github/workflow/status/nodejs/llnode/on%20push%20or%20pull_request/master?style=flat-square

Happy to just make this change myself if it's too much bother but wanted to capture your work if you are still interested.

No9 avatar Sep 16 '22 22:09 No9