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

Process else if using ElseIfNode. #2663

Merged
merged 2 commits into from
Dec 28, 2022
Merged

Process else if using ElseIfNode. #2663

merged 2 commits into from
Dec 28, 2022

Conversation

nojaf
Copy link
Contributor

@nojaf nojaf commented Dec 23, 2022

Related to #2449.
@dawedawe this might seem a bit complex and it definitely is a very creative way to approach the problem space.

In essence, I wrote a special type that deals with the trivia assignment of else if.
If a trivia would be assigned to either else or if, the best possible node is selected so that we can write else if in one piece without trivia interference.

This isn't without drawbacks but given the delicate nature this might do the trick.

@nojaf nojaf requested a review from dawedawe December 23, 2022 15:39
@nojaf nojaf marked this pull request as ready for review December 28, 2022 08:01
@nojaf nojaf merged commit 81cc6d8 into fsprojects:v5.2 Dec 28, 2022
@nojaf nojaf deleted the poc-2449 branch December 28, 2022 08:01
nojaf added a commit that referenced this pull request Jan 2, 2023
* Process `else if` using ElseIfNode.

* Rename x to _.
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

Successfully merging this pull request may close these issues.

2 participants