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

Use emoji to illustrate good and bad example commit messages #1235

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Dec 1, 2023

Makes it clear which are the good and bad examples.

✅ Doesn't rely on colour: colour blind people can still see a tick or cross
✅ We don't rely on them: the good/bad meaning is still clear from the text

Details

image

Details

image

Compare, for example, the Google developer docs: https://developers.google.com/style/ui-elements

Also fix some formatting.


📚 Documentation preview 📚: https://cpython-devguide--1235.org.readthedocs.build/

@hugovk hugovk merged commit 9f2b215 into python:main Dec 1, 2023
3 checks passed
@ezio-melotti
Copy link
Member

PDF builds used to have issues with emojis, and even other Unicode characters.
I don't know if that's still the case and we don't seem to build PDFs for the devguide, so in this case it should be ok.
This is something to keep in mind if you are planning to do similar changes to the cpython docs though.

@hugovk hugovk deleted the good-bad branch December 3, 2023 20:52
@hugovk
Copy link
Member Author

hugovk commented Dec 3, 2023

Good reminder. There's also the epub builds for CPython (but I think they might be broken).

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.

3 participants