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

Avoid space around pow for None, True and False #8189

Merged
merged 2 commits into from
Oct 25, 2023
Merged

Conversation

MichaReiser
Copy link
Member

Summary

Closes #8187

Test Plan

Added integration test.

The similarity index doesn't change

@MichaReiser MichaReiser requested a review from konstin October 24, 2023 23:33
@MichaReiser
Copy link
Member Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@MichaReiser MichaReiser added the formatter Related to the formatter label Oct 24, 2023
@zanieb
Copy link
Member

zanieb commented Oct 25, 2023

I'm not really sure I prefer this formatting, but I am not opposed.

@MichaReiser
Copy link
Member Author

I'm not really sure I prefer this formatting, but I am not opposed.

The removal of the space in general or more specifically removing the space when either side is False, True, or None?

@MichaReiser MichaReiser merged commit e36afc3 into main Oct 25, 2023
17 checks passed
@MichaReiser MichaReiser deleted the power-op-spacing branch October 25, 2023 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
formatter Related to the formatter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unintentional formatter deviation: spaces added around power operator (**)
2 participants