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

Format \N unicode escapes #2828

Closed
TomFryers opened this issue Jan 29, 2022 · 0 comments · Fixed by #2916
Closed

Format \N unicode escapes #2828

TomFryers opened this issue Jan 29, 2022 · 0 comments · Fixed by #2916
Labels
T: style What do we want Blackened code to look like?

Comments

@TomFryers
Copy link
Contributor

Describe the style change

Black could have an opinion of the correct case for \N unicode escapes. For examples, formatting \N{ox} to \N{OX}.

Examples in the current Black style

# This is not nice. All oxen should look the same.
print("\N{ox}\N{OX}")

Desired style

# Now the oxen will be happier.
print("\N{OX}\N{OX}")

Upper-case is standard for these, I believe. (For example, in the Python docs.)

This is related to issue number 2067.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: style What do we want Blackened code to look like?
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant