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

[DES] parse by indent level #6

Open
Rekihyt opened this issue Sep 12, 2023 · 1 comment
Open

[DES] parse by indent level #6

Rekihyt opened this issue Sep 12, 2023 · 1 comment

Comments

@Rekihyt
Copy link
Owner

Rekihyt commented Sep 12, 2023

Describe the change

Change parsing from a line by line approach to greedily accept lines as one App until indentation levels are the same.

Reason(s) for it to be added

Multi-line Apps, i.e. lists or patterns

@Rekihyt
Copy link
Owner Author

Rekihyt commented Feb 10, 2024

Don't use whitespace for parsing except in two cases:

  • A single newline separates apps, unless they are operators or commas with empty second args
  • A double newline separates everything.

Instead, indents will indicate parsing after the fact by the linter. If two lines are special cased into one expression, the linter will indent the second line.

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

No branches or pull requests

1 participant