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

Missing support for Python3 features #58

Closed
5 of 6 tasks
soupytwist opened this issue Mar 21, 2019 · 7 comments
Closed
5 of 6 tasks

Missing support for Python3 features #58

soupytwist opened this issue Mar 21, 2019 · 7 comments
Assignees

Comments

@soupytwist
Copy link
Collaborator

soupytwist commented Mar 21, 2019

These AST nodes are not supported and can cause parse failures:

  • Constant
  • JoinedStr/FormattedValue
  • YieldFrom
  • AsyncFunctionDef
  • AsyncFor
  • AsyncWith

Will update this list as support is added.

@mmv
Copy link

mmv commented May 22, 2019

Also missing support for PEP 448 - Additional Unpacking Generalizations

For example merged = {**a, **b} fails with a cryptic error

  File "pasta/base/annotate.py", line 804, in visit_Dict
    self.visit(key)
  File "pasta/base/annotate.py", line 1189, in visit
    fmt.set(node, 'indent', self._indent)
  File "pasta/base/formatting.py", line 37, in set
    _formatting_dict(node)[name] = value
  File "pasta/base/formatting.py", line 53, in _formatting_dict
    return getattr(node, PASTA_DICT)
AttributeError: 'NoneType' object has no attribute '__pasta__'

because the node key in visit_Dict will be None

@soupytwist
Copy link
Collaborator Author

Also missing support for PEP 448 - Additional Unpacking Generalizations

Thanks, split this into #64 to limit the scope of this bug.

@tapaswenipathak
Copy link

tapaswenipathak commented Aug 3, 2019

Hi folks: Can I take adding YieldFrom or is the ticket internal?

@soupytwist
Copy link
Collaborator Author

Hi folks: Can I take adding YieldFrom or is the ticket internal?

Please do!

@tapaswenipathak
Copy link

@soupytwist: Is there a channel where I can ask few questions or would you be ok w/ I creating a draft PR and you having a look if I understand well architecture wise?

@soupytwist
Copy link
Collaborator Author

I don't have a good channel for it, no. Feel free to send a draft PR for review or ask anything here. I've been meaning to add some instructions on testing locally, will do that today.

@soupytwist
Copy link
Collaborator Author

These have all been addressed already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants