-
Notifications
You must be signed in to change notification settings - Fork 42
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
elif has wrong indentation #2
Labels
Comments
jwhonce
added a commit
to jwhonce/beautysh
that referenced
this issue
Jun 9, 2017
Fixes lovesegfault#2 Signed-off-by: Jhon Honce <jhonce@redhat.com>
Merged
Closed
trystero11
added a commit
to trystero11/beautysh
that referenced
this issue
May 3, 2021
Make the ad-hoc oudent (added in lovesegfault#2) apply only when `elif` is followed by `then` on the same line. In my testing, this addresses lovesegfault#57 without breaking the lovesegfault#2 test case. I'm sure the regex could be cleaned up, though.
trystero11
pushed a commit
to trystero11/beautysh
that referenced
this issue
May 3, 2021
trystero11
added a commit
to trystero11/beautysh
that referenced
this issue
May 3, 2021
`elif` test case to address both lovesegfault#2 and lovesegfault#57.
lovesegfault
pushed a commit
that referenced
this issue
May 3, 2021
trystero11
added a commit
to trystero11/beautysh
that referenced
this issue
May 3, 2021
…vesegfault#57) Make the ad-hoc oudent (added in lovesegfault#2) apply only when `elif` is followed by `then` on the same line. In my testing, this addresses lovesegfault#57 without breaking the lovesegfault#2 test case. Test case to be added in lovesegfault#76
lovesegfault
pushed a commit
that referenced
this issue
May 3, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If you have something like:
The result is:
So it seems elif is not handled correctly.
The text was updated successfully, but these errors were encountered: