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

Meta Issue for Future Parser Changes #2834

Open
6 tasks
Rowlando13 opened this issue Jan 4, 2025 · 6 comments
Open
6 tasks

Meta Issue for Future Parser Changes #2834

Rowlando13 opened this issue Jan 4, 2025 · 6 comments

Comments

@Rowlando13
Copy link
Collaborator

Rowlando13 commented Jan 4, 2025

We are not adding anything to #2205. This is for changes after #2205 is merge.

We need to have a grasp on the parser as a whole. For each requested new parser change we need:

  • a minimal example of the desired behavior
  • can it be done with current parser
  • If it can be, it is documented

Issues already slated for addition in parser rewrite are not included in the list.

In-Progress Rewrite

Preparation for Future Additions

Issues

@Rowlando13 Rowlando13 changed the title ? Meta Issue for Parser Jan 4, 2025
@Rowlando13
Copy link
Collaborator Author

@kdeldycke @janluke @ofek Feel free to request adds. I went through open issues to generate this, but I know many were opened and closed. I am also a bit unclear on what is and is not the parser, so the list may not be exactly right.

@Rowlando13
Copy link
Collaborator Author

@AndreasBackx Let me know what you think.

@davidism
Copy link
Member

davidism commented Jan 8, 2025

This issue already tracks many of the issues related to the parser rewrite: #2205

@kdeldycke
Copy link
Contributor

@Rowlando13 I'll have to revisit my feature list and hacks in light of the new architecture anyway. And I don't want to add too much constraints that can slow down that effort. So I would say to just go ahead with the rewrite of the parser! 😃 I'll catch up later! 😁

@Rowlando13
Copy link
Collaborator Author

Yeah, this is for future work. I did not want to add anything to #2205 since it was already in progress and seemed large.

@Rowlando13
Copy link
Collaborator Author

Rowlando13 commented Jan 8, 2025

I will have to change the wording. I meant this to be for stuff after the current rewrite.

@Rowlando13 Rowlando13 changed the title Meta Issue for Parser Meta Issue for Parser Future Parser Changes Jan 8, 2025
@Rowlando13 Rowlando13 changed the title Meta Issue for Parser Future Parser Changes Meta Issue for Future Parser Changes Jan 8, 2025
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