Skip to content
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

Refresh CoverAlls badge image #91

Closed
wants to merge 1 commit into from
Closed

Refresh CoverAlls badge image #91

wants to merge 1 commit into from

Conversation

kdh0102
Copy link

@kdh0102 kdh0102 commented Nov 12, 2021

This PR resolves the error with CoverAlls badge in the README.md file.

Check the branch page below:
https://github.com/swsnu/swpp2021-team4/tree/coveralls-badge

You can see the badge image has been updated.

Check the issue for the reference:
lemurheavy/coveralls-public#971 (comment)

@kdh0102 kdh0102 requested a review from young924 November 12, 2021 14:08
@yeppin
Copy link
Contributor

yeppin commented Nov 13, 2021

@kdh0102 Thank you for your advice.
We figured out that there was a problem rendering svg file on markdown due to a sort of security concept.
We recently fixed the problem by adding &sanitize=true on url.
Please refer to this document https://stackoverflow.com/questions/46381436/github-svg-not-rendering-at-all

@kdh0102 kdh0102 closed this Nov 13, 2021
@kdh0102 kdh0102 deleted the coveralls-badge branch November 13, 2021 07:13
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