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.

Sign up for GitHub

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

Jump to bottom

Standardize case for string prefixes #198

Closed
JelleZijlstra opened this issue May 9, 2018 · 2 comments
Closed

Standardize case for string prefixes #198

JelleZijlstra opened this issue May 9, 2018 · 2 comments
Labels
T: enhancement New feature or request

Comments

@JelleZijlstra
Copy link
Collaborator

In implementing a fix for #188 it occurred to me that black could also standardize the case of string literal prefixes (presumably to lowercase). For example, change R"string" to r"string".

@ambv is this something you'd be open to doing?
@ambv
Copy link
Collaborator

ambv commented May 9, 2018

Yes, that's an obvious omission from our previous standardization. Fortunately way fewer people actually use upper-case string prefixes :)

cc @zsol

@ambv ambv added the T: enhancement New feature or request label May 9, 2018
@JelleZijlstra JelleZijlstra mentioned this issue May 9, 2018
Merged
@JelleZijlstra
Copy link
Collaborator Author

Sounds good, I can implement this in #199.

@ambv ambv closed this as completed May 9, 2018
@janosh janosh mentioned this issue Jan 23, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ambv @JelleZijlstra