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

Fix invalid/deprecated escape sequences #63

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

agners
Copy link
Contributor

@agners agners commented Aug 14, 2024

There are a couple more invalid escape sequences. These are not meant to be Python string escape sequences, but literal backspace used for those regular expressions. Use the raw string prefix r"" for these strings.

@creekorful creekorful self-requested a review August 14, 2024 12:09
@creekorful creekorful self-assigned this Aug 14, 2024
@creekorful creekorful added the bug label Aug 14, 2024
@agners agners force-pushed the fix-invalid-escape-sequences-2 branch from b8b07d3 to ae75ebb Compare September 4, 2024 20:01
There are a couple more invalid escape sequences. These are not meant to
be Python string escape sequences, but literal backspace used for those
regular expressions. Use the raw string prefix r"" for these strings.
@agners agners force-pushed the fix-invalid-escape-sequences-2 branch from ae75ebb to 1653638 Compare September 4, 2024 20:06
@agners
Copy link
Contributor Author

agners commented Sep 4, 2024

I've fixed a couple more SyntaxWarning.

@agners
Copy link
Contributor Author

agners commented Oct 2, 2024

@creekorful gentle ping on this one. Would be nice to get rid of some more deprecation warnings 😄

@creekorful creekorful merged commit 1dc8483 into nilp0inter:main Oct 2, 2024
@creekorful
Copy link
Collaborator

@agners Thanks for your contribution!

@creekorful creekorful added this to the v1.3.1 milestone Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants