Skip to content

fix GitHub markdown rendering by removing space from badge style #774

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

Merged
merged 1 commit into from
Apr 11, 2025

Conversation

spencerschrock
Copy link
Member

Related to ossf/scorecard#4591

While the redirect was working locally with curl, I suspect the space may be causing issues with GitHub's camo proxy. This suspicion is due to a test README I did which directly linked failed to render:

![image](https://img.shields.io/<...>label=openssf scorecard<...>

But using the plus seemed to render.

![image](https://img.shields.io/<...>label=openssf+scorecard<...>

While the redirect was working locally with curl, I suspect the space
may be causing issues with GitHub's camo proxy. This suspicion is due to
a test README I did which directly linked failed to render:

    ![image](https://img.shields.io/<...>label=openssf scorecard<...>

But using the plus seemed to render.

    ![image](https://img.shields.io/<...>label=openssf+scorecard<...>

Signed-off-by: Spencer Schrock <[email protected]>
@spencerschrock spencerschrock requested review from a team, justaugustus and raghavkaul and removed request for a team April 11, 2025 14:52
Copy link

netlify bot commented Apr 11, 2025

Deploy Preview for ossf-scorecard canceled.

Name Link
🔨 Latest commit 9fd7a47
🔍 Latest deploy log https://app.netlify.com/sites/ossf-scorecard/deploys/67f92cb60c2a880008f3e935

@spencerschrock
Copy link
Member Author

the failure to render in my small test is probably just due to the space, but this seems like a harmless change to make to test once it gets deployed to the staging instance

@spencerschrock spencerschrock merged commit d67807b into ossf:main Apr 11, 2025
11 checks passed
@spencerschrock spencerschrock deleted the badge branch April 11, 2025 16:20
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