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

pyproject.toml: Configure Black to skip-string-normalization #6624

Merged
merged 3 commits into from
Jun 8, 2022

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Jun 3, 2022

Minimal configuration to ensure that Black does not convert quotes.
Black does not support reading its configuration from setup.cfg.
This file can be deleted when we convert to favoring double quotes.
https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#configuration-via-a-file

Closes #

Technical

Testing

Screenshot

Stakeholders

Minimal configuration to ensure that Black does not convert quotes.
Black does not support reading its configuration from `setup.cfg`.
This file can be deleted when we convert to favoring double quotes.
https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html
@cclauss cclauss requested a review from cdrini June 3, 2022 04:24
@cclauss cclauss added Theme: Development Issues related to the developer experience and the dev environment. [managed] Affects: Developers labels Jun 3, 2022
Copy link
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woohoo, nice! Love that pyproject.toml is a PEP standard, too :) Removed any refs to --skip-string-normalization now that it's the default. This will also make configuring VS code easier!

@cdrini cdrini merged commit cb6f5c1 into master Jun 8, 2022
@cdrini cdrini deleted the pyproject.toml_Black_—-skip-string-normalization branch June 8, 2022 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: Developers Theme: Development Issues related to the developer experience and the dev environment. [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants