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

Full diff issue #11688

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Full diff issue #11688

wants to merge 14 commits into from

Conversation

JamieC2002
Copy link

Closes #10863

Resolved issue in Full DIff Output with string lists where strings that were implicitly concatenated/long strings were treated broken up into multiple strings, causing the output format to be misleading. Now, implicitly concatenated strings/long strings are treated as one string, and the output is consistent with other scenarios involving strings.

Test case:

def test() -> None:
 // Implicit concatenated string 
 ex = [
     "This is some dummy test which shows the strange way in which Pycharm"
     " displays the full diff."
 ]
 assert ex == []

Before:
Screenshot 2023-12-07 at 7 51 56 PM

string in ex is output as two strings.

After:
Screenshot 2023-12-07 at 7 52 42 PM

string in ex is now output as one string.

Worked with @kayliny02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Full diff for list of strings wrong for multi-line string
2 participants