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

Update README.md #311

Closed
wants to merge 1 commit into from
Closed

Update README.md #311

wants to merge 1 commit into from

Conversation

f-necas
Copy link
Contributor

@f-necas f-necas commented Nov 5, 2024

No description provided.

@f-necas f-necas requested a review from edevosc2c November 5, 2024 10:00
@edevosc2c
Copy link
Member

I have chosen the unicode character because you have the warning icon correctly displayed if you open the file using nano or vim.

@f-necas
Copy link
Contributor Author

f-necas commented Nov 7, 2024

But it displays as shit in github. It's supposed to catch attention at first read.

@edevosc2c
Copy link
Member

I think people will notice it, it's literally written "if that doesn't work". And it's after the guide steps not before so people will read the steps before noticing the warning icon anyway.

@f-necas f-necas closed this Nov 8, 2024
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