-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Improve long values in dict literals #3440
Conversation
diff-shades results comparing this PR (ba1b968) to main (a282181). The full diff is available in the logs under the "Generate HTML diff report" step.
|
Notes from diff-shades:
Note that both the key and the value in the dictcomp use Pandas has a similarly odd-looking case with a
Otherwise I like the changes from the PR: they make it easier to see when one dict key ends and the next one begins, because values get indented. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, just noticed two typos
Co-authored-by: Jelle Zijlstra <[email protected]>
Co-authored-by: Jelle Zijlstra <[email protected]>
Thanks for the review! Agreed above two examples are out of scope from this PR, they concern the dict keys. Opened #3442 separately. |
Description
This PR changes the preview style:
Usually I'd like to keep the PR's scope minimal, but the lambda change & dict value change are depended by each other so it's hard to split into two PRs.
Checklist - did you ...
CHANGES.md
if necessary?