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

Fix syntax warning in docstring #3056

Merged
merged 1 commit into from Jan 6, 2019
Merged

Fix syntax warning in docstring #3056

merged 1 commit into from Jan 6, 2019

Conversation

tirkarthi
Copy link
Contributor

This fixes the SyntaxWarning introduced in CPython master. This is similar to pallets/jinja#646 where DeprecationWarning is now converted to SyntaxWarning. I have added r to convert the docstring as raw string that fixes the issue.

Before patch warning :

flask/app.py:1693: SyntaxWarning: invalid escape sequence \s

Relevant CPython issue : https://bugs.python.org/issue32912

@BoboTiG BoboTiG mentioned this pull request Jan 4, 2019
Closed
@davidism davidism changed the base branch from master to 1.0-maintenance January 6, 2019 18:09
@davidism davidism merged commit 10d0ed6 into pallets:1.0-maintenance Jan 6, 2019
@davidism davidism added the docs label Jan 6, 2019
@davidism davidism added this to the 1.0.3 milestone Jan 6, 2019
@tirkarthi tirkarthi deleted the fix-syntax-warning branch January 6, 2019 18:37
@AbdealiLoKo AbdealiLoKo mentioned this pull request Aug 4, 2019
Closed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
docs
Projects
None yet
Milestone
1.0.3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tirkarthi @davidism