Skip to content

Conversation

rootulp
Copy link
Contributor

@rootulp rootulp commented Aug 31, 2021

It looks like CircleCI badges generated via these steps don't render correctly in this README. My suspicion is that the . in set.js is a possible cause.

I generated a shield badge via https://shields.io/category/build and it works in the preview

It looks like CircleCI badges generated via [these steps](https://circleci.com/docs/2.0/status-badges/#creating-badges-for-private-repositories96166267374796c653d736869656c64) don't render correctly in this README. My suspicion is that the `.` in set.js is a possible cause.

I generated a shield badge via https://shields.io/category/build and it works in the preview
@rootulp rootulp mentioned this pull request Aug 31, 2021
@cgewecke cgewecke self-requested a review August 31, 2021 02:33
Copy link
Contributor

@cgewecke cgewecke left a comment

Choose a reason for hiding this comment

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

All right! Yeah, makes sense. Definitely works

@rootulp
Copy link
Contributor Author

rootulp commented Aug 31, 2021

Potentially no longer necessary because attempt 1 appears to work now (?!)

Screen Shot 2021-08-30 at 10 33 15 PM

Is it plausible that CircleCI was not providing this image until shields.io fetched it? I'm confused.

@rootulp rootulp closed this Aug 31, 2021
@cgewecke
Copy link
Contributor

Hahahah!!! I have no idea. I'm going to re-open this and merge because this seems more stable and all the other links use the same service.

@cgewecke cgewecke reopened this Aug 31, 2021
@cgewecke cgewecke merged commit cd29648 into SetProtocol:master Aug 31, 2021
@rootulp rootulp deleted the patch-1 branch August 31, 2021 02:41
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.

2 participants