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

Fix broken icons by using pure HTML instead #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ausarb
Copy link

@ausarb ausarb commented Dec 22, 2022

fixes #4

@ausarb
Copy link
Author

ausarb commented Dec 22, 2022

You can see what this would like like by looking at a sample PR on my fork here: ausarb#1

Run that generated the PR comments: https://github.com/ausarb/describe-cfn-change-set/actions/runs/3761430922/jobs/6393176380

@ausarb ausarb changed the title Fix broken icons by using MD supported emojis instead Fix broken icons by using pure HTML instead Dec 23, 2022
@ausarb
Copy link
Author

ausarb commented Dec 23, 2022

After thinking about this, I replaced the :foo: emojis with HTML codes. They render the same in the PR but will now work anywhere.

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.

Icons on the changeset table are broken due to HTTP 403 errors
1 participant