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
Update various regex escape sequences #2378
Comments
If there is no one working on this, I would like to. Can you tell steps to duplicate the issue please? |
@PabloDino Install Python 3.6.8 or later and try to import every module. The fix the regex either by using raw strings or using proper escape such that this works both on Python 2 and 3 |
I'm on it- been working through some exercises but not seeing any warnings. Can you post a code snippet in which the warnings manifest pl |
|
And FWIW: https://docs.python.org/3/reference/lexical_analysis.html#string-and-bytes-literals
|
$ python --version
|
I'm seeing the same output as @pombredanne. |
Hi, is @PabloDino still planning to work on the issue? I have been able to replicate @pombredanne 's output and would like to work on fixing this issue. |
Go ahead, I haven't replicated yet
…On Mon, Sep 30, 2019 at 11:40 AM Armin Stepanjan ***@***.***> wrote:
Hi, is @PabloDino <https://github.com/PabloDino> still planning to work
on the issue?
I have been able to replicate @pombredanne
<https://github.com/pombredanne> 's output and would like to work on
fixing this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2378?email_source=notifications&email_token=ABRSN4KL27M5TYFOR65HRMDQMIMVPA5CNFSM4IRCRGM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76DA2A#issuecomment-536621160>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABRSN4KRASPRV6I4VLHFNILQMIMVPANCNFSM4IRCRGMQ>
.
|
@ab-10 Have you been able to fix those dep warnings? |
An updated list with Python 3.8 with running below command :
|
@gertjanwytynck I'm currently fixing them one by one, should be done by the end of the week. |
Has this been completed? |
It looks like there are still a few left. I wonder if adding a unit test could help.
... and even though the impact of tools deprecation is not much, there is a bit of irony that the
|
The latest versions of Python are more strict wrt. escape in regex.
For instance with 3.6.8, there are 10+ warnings like this one:
The regex(es) should be updated to silence these warnings.
The text was updated successfully, but these errors were encountered: