Visitar URL original
gh-105042: Disable unmatched parens syntax error in python tokenize by lysnikolaou · Pull Request #105061 · python/cpython · GitHub
Skip to content

gh-105042: Disable unmatched parens syntax error in python tokenize - #105061

Merged
pablogsal merged 3 commits into
python:mainfrom
lysnikolaou:tokenize-unmatched-parens
May 30, 2023
Merged

gh-105042: Disable unmatched parens syntax error in python tokenize#105061
pablogsal merged 3 commits into
python:mainfrom
lysnikolaou:tokenize-unmatched-parens

Conversation

@lysnikolaou

@lysnikolaou lysnikolaou commented May 29, 2023 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

@bedevere-bot bedevere-bot mentioned this pull request May 29, 2023
Closed
pablogsal
pablogsal reviewed May 30, 2023
View reviewed changes
Comment thread Python/Python-tokenize.c Outdated
@pablogsal

pablogsal commented May 30, 2023

Copy link
Copy Markdown
Member

We also need a bunch of tests that cover these, one in the inspect module and the other in the tokenize module

pablogsal
pablogsal reviewed May 30, 2023
View reviewed changes
Comment thread Lib/tokenize.py Outdated
pablogsal
pablogsal reviewed May 30, 2023
View reviewed changes
Comment thread Parser/tokenizer.c Outdated
@lysnikolaou
lysnikolaou force-pushed the tokenize-unmatched-parens branch from 049da25 to ed1237a Compare May 30, 2023 09:55
@lysnikolaou
lysnikolaou marked this pull request as ready for review May 30, 2023 09:56
@lysnikolaou

lysnikolaou commented May 30, 2023

Copy link
Copy Markdown
Member Author

@pablogsal This is ready for another review.

@pablogsal

pablogsal commented May 30, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

We also need a bunch of tests that cover these, one in the inspect module and the other in the tokenize module

Can you add a couple of tests for this?

Maybe something on these lines: #105042 (comment) in the inspect module here:

cpython/Lib/test/test_inspect.py

Line 779 in 0656d23

def test_parenthesized_multiline_lambda(self):

@lysnikolaou

lysnikolaou commented May 30, 2023

Copy link
Copy Markdown
Member Author

@pablogsal Thanks for the reminder! Tests added in c7481a5.

pablogsal
pablogsal approved these changes May 30, 2023
View reviewed changes

@pablogsal pablogsal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Fantastic work!

@pablogsal pablogsal added the needs backport to 3.12 only security fixes label May 30, 2023
@pablogsal
pablogsal merged commit 70f315c into python:main May 30, 2023
@miss-islington

miss-islington commented May 30, 2023

Copy link
Copy Markdown
Contributor

Thanks @lysnikolaou for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 30, 2023
@lysnikolaou @miss-islington
pythongh-105042: Disable unmatched parens syntax error in python toke…
bb899c1 
…nize (pythonGH-105061)

(cherry picked from commit 70f315c)

Co-authored-by: Lysandros Nikolaou <lisandrosnik@gmail.com>
@bedevere-bot

bedevere-bot commented May 30, 2023

Copy link
Copy Markdown

GH-105120 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label May 30, 2023
pablogsal pushed a commit that referenced this pull request May 31, 2023
@miss-islington @lysnikolaou
[3.12] gh-105042: Disable unmatched parens syntax error in python tok…
2f8c22f 
…enize (GH-105061) (#105120)

gh-105042: Disable unmatched parens syntax error in python tokenize (GH-105061)
(cherry picked from commit 70f315c)

Co-authored-by: Lysandros Nikolaou <lisandrosnik@gmail.com>
@erlend-aasland erlend-aasland mentioned this pull request Jun 6, 2023
Closed
@asottile asottile mentioned this pull request Jun 13, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pablogsal pablogsal pablogsal approved these changes

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lysnikolaou @pablogsal @miss-islington @bedevere-bot