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

Merge Task into Task2 #11

Open
3 tasks
thilp opened this issue Feb 5, 2019 · 0 comments
Open
3 tasks

Merge Task into Task2 #11

thilp opened this issue Feb 5, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@thilp
Copy link
Member

thilp commented Feb 5, 2019

TIP/transformer#10Arbitrarily nested scenarios” solved TIP/docs#330Introduce generic multi-level nested scenarios in Transformer” by introducing the transformer.python module, which represents Python code as a syntax tree (i.e. a tree of objects, each representing a syntactic element like "class" or "assignment").

transformer.python.OpaqueBlock does not represent a regular Python element but encapsulates a string and "pretends" it is a normal syntax tree. It helps by not forcing the introduction of the syntax tree to instantly propagate everywhere in Transformer: without OpaqueBlock, Task and all plugins would have had to be updated in TIP/transformer#10.

However, eventually this usage of OpaqueBlock should go away: plugins should be updated to use the more powerful & testable framework of transformer.python too. Once plugins are updated, Task can be merged into the Task2 proxy. At that point, the only remaining usage of OpaqueBlock should be for cases not yet covered by the rest of transformer.python (like the raise statement today).

ℹ️ Issue imported from TIP/docs#395

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant