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

Warn when an escaped newline skips multiple lines #87671

Merged
merged 4 commits into from
Aug 12, 2021
Merged

Warn when an escaped newline skips multiple lines #87671

merged 4 commits into from
Aug 12, 2021

Commits on Aug 11, 2021

  1. Warn when an escaped newline skips multiple lines

    @jesyspa
    jesyspa committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    a03fbfe View commit details
    Browse the repository at this point in the history

  2. Update format string tests to explicitly escape multiple newlines 

    From what I can tell, the goal of the tests is to ensure that the error
formatting is correct.  I think this is still being tested as intended
after this change.
    @jesyspa
    jesyspa committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    2dff700 View commit details
    Browse the repository at this point in the history

  3. Add UI tests for string escape warnings.

    @jesyspa
    jesyspa committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    efe069c View commit details
    Browse the repository at this point in the history

  4. Renamed variable str -> tail for clarity

    @jesyspa
    jesyspa committed Aug 11, 2021
    Configuration menu
    Copy the full SHA
    07aacf5 View commit details
    Browse the repository at this point in the history