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.

Sign up for GitHub

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

Jump to bottom

Implement Definition API for new AST #5442

Closed
wdanilo opened this issue Feb 5, 2023 · 0 comments · Fixed by #4113
Closed

Implement Definition API for new AST #5442

wdanilo opened this issue Feb 5, 2023 · 0 comments · Fixed by #4113
Assignees
@kazcw
Labels
-parser p-low Low priority x-new-feature Type: new feature request

Comments

@wdanilo
Copy link
Member

wdanilo commented Feb 5, 2023

This task is automatically imported from the old Task Issue Board and it was originally created by Keziah Wesley.
Original issue is here.

Use the new AST types to reimplement the double_representation::definition API.

Also, rewrite the collapse refactoring, as it's the consumer of much of this API.
@sylwiabr sylwiabr removed this from Issues Board Feb 8, 2023
@kazcw kazcw linked a pull request Feb 10, 2023 that will close this issue
Port graph editor to new AST #4113
Merged
4 tasks
@mergify mergify bot closed this as completed in #4113 Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kazcw kazcw

Labels
-parser p-low Low priority x-new-feature Type: new feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Port graph editor to new AST enso-org/enso
2 participants
@kazcw @wdanilo