-
-
Notifications
You must be signed in to change notification settings - Fork 39.7k
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
[Enhancement] Prepare for SyntaxWarning
#22562
Conversation
Thank you for your contribution! |
Busy week + internet was down for most part of it. Sorry for the delay I think i've fixed everything now, running a |
Description
Is not really an issue yet, but invalid escape sequences, eg
'\['
will throw aSyntaxWarning
on newer versions of Python. To avoid them i've looked throughlib/python
and fixed the faulty ones i could find.Used
git diff
to check the output of both changes and didn't seem to affect functionality.PS: Saw a couple places with stuff like
'\\\\'
which could becomer'\\'
, but decided not to make more changes than strictly needed.PS2: Bux could've been fixed by
'\B'
->'\\B'
but i decided to make it ar""
to change the'\\"'
and get the right border to alignTypes of Changes
Issues Fixed or Closed by This PR
Checklist