Author mbussonn
Recipients benjamin.peterson, georg.brandl, inada.naoki, jeff.allen, mbussonn, serhiy.storchaka, vstinner
Date 2017-02-22.16:53:35
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1487782415.52.0.647582230183.issue29463@psf.upfronthosting.co.za>
In-reply-to
Content
thank you for your work on the AST, I know many developers are looking forward to improvement and stabilisation with the hope of having it stable (and documented) in the stdlib at some point. 

The recent change in PR 46 now change (at least) the constructor of `ast.Module` to take a second mandatory parameter (the docstring). 

I know the ast is autogenerated and "use at your own risk". But IPython for example, use `mod = ast.Module([nodes])`, with the second mandatory parameter added to Module that make it incompatible with current Python 3.7. 
Well it's long until it's released, and we can patch things, but I'm sure we are not the only one in this case, and we'd like older version of IPython to still be compatible with Python 3.7, so if  `Module()`'s second parameter (the docstring) could be optional, that would be great. 

I would be happy if it raise a deprecation warning that it will be required in the future.

I'm of course speaking about `Module` because that's the first error I encountered, but I'm guessing it applies to other changed AST nodes. 

Thanks.
History
Date User Action Args
2017-02-22 16:53:35mbussonnsetrecipients: + mbussonn, georg.brandl, vstinner, benjamin.peterson, inada.naoki, serhiy.storchaka, jeff.allen
2017-02-22 16:53:35mbussonnsetmessageid: <1487782415.52.0.647582230183.issue29463@psf.upfronthosting.co.za>
2017-02-22 16:53:35mbussonnlinkissue29463 messages
2017-02-22 16:53:35mbussonncreate