[issue39969] Remove Param expression context from AST

2020-03-19 Thread Batuhan Taskaya


Batuhan Taskaya  added the comment:

> Batuhan: You maybe want to propose a fix there. It shouldn't be hard to fix 
> the issue.

Serhiy has an open PR about adding some dummy classes that will represent 
recent deletions like Suite, Param and AugStore/Load. 
https://github.com/python/cpython/pull/19056

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue39969] Remove Param expression context from AST

2020-03-19 Thread STINNER Victor


STINNER Victor  added the comment:

FYI this change broke the chameleon project:
https://github.com/malthe/chameleon/issues/303

Batuhan: You maybe want to propose a fix there. It shouldn't be hard to fix the 
issue.

--
nosy: +vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue39969] Remove Param expression context from AST

2020-03-15 Thread Pablo Galindo Salgado

Pablo Galindo Salgado  added the comment:


New changeset 8689209e0338943dba9b7ff5566b8a420374764c by Batuhan Taşkaya in 
branch 'master':
bpo-39969: Remove ast.Param node class as is no longer used (GH-19020)
https://github.com/python/cpython/commit/8689209e0338943dba9b7ff5566b8a420374764c


--
nosy: +pablogsal

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue39969] Remove Param expression context from AST

2020-03-15 Thread Pablo Galindo Salgado


Change by Pablo Galindo Salgado :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue39969] Remove Param expression context from AST

2020-03-15 Thread Batuhan Taskaya


Change by Batuhan Taskaya :


--
keywords: +patch
pull_requests: +18369
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/19020

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue39969] Remove Param expression context from AST

2020-03-15 Thread Batuhan Taskaya


New submission from Batuhan Taskaya :

Param is an expression context that is no longer in use, we can simply remove 
it. This node predates the arguments node, and if I am not misguessing used 
inside of function signatures.

--
components: Library (Lib)
messages: 364238
nosy: BTaskaya
priority: normal
severity: normal
status: open
title: Remove Param expression context from AST
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com