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

No indent after complex hotkey, hotstring #303

Closed
kyklish opened this issue Dec 26, 2022 · 3 comments · Fixed by #304
Closed

No indent after complex hotkey, hotstring #303

kyklish opened this issue Dec 26, 2022 · 3 comments · Fixed by #304
Assignees
Labels
bug something isn't working formatter code formatter issues

Comments

@kyklish
Copy link
Contributor

kyklish commented Dec 26, 2022

Expected formatted output snippet

F1 & F2 Up::
    code
return

::btw::
    code
return

Actual formatted output snippet

F1 & F2 Up::
code
return

::btw::
code
return
@mark-wiemer
Copy link
Member

@kyklish are you sure the expectation is to indent after the hotstring?

::btw::by the way
    code
return

PR #304 explicitly adds a test to keep the code not indented

@CoffeeChaton
Copy link

#Warn All

; no problem
::ahk::autohotkey 

; also no problem
::text::
MsgBox, % "text" 
return

; but...
::btw::by the way
MsgBox, % "text" ; open "#Warn, All" -> Warning: This line will never execute, due to Return preceding it.
return

; or can't run of this case
;
; ::btw2::by the way
; by the way2
; return

Did I miss something?

@kyklish
Copy link
Contributor Author

kyklish commented Jan 30, 2023

@kyklish are you sure the expectation is to indent after the hotstring?

::btw::by the way
    code
return

PR #304 explicitly adds a test to keep the code not indented

My typo in bug description. Got fix first message.

@CoffeeChaton

Did I miss something?

It's my mistake. Look at fixed first message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working formatter code formatter issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants