-
Notifications
You must be signed in to change notification settings - Fork 11
Package creation: initial version #58
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
base: lamdera-next
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,3 @@ | |||
{ | |||
"elmLS.elmFormatPath": "elm-format-hack" | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Accidental commit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nooot really? The Do
helpers for elm-pages
kinda require elm-format-hack
to produce sensible formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah right I see. That makes me like the <| \_ ->
incantation even less compared to the passthrough variant approach we use in lamdera/script... first person to edit the script is going to get a surprise when the formatting kicks in 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you deal with using the result of previous steps usually? To be fair, this script mostly does <| \_ ->
, where a passthrough
would work, but what if you actually care about the result instead of ignoring it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've got examples in runtime builds – I'll DM you.
Can you drop a readme with the incantation required to run this as well? :) |
Added a minimal comment. There isn't much to say, tbh, no CLI flags, no options, no environment vars to set |
Shall we merge? |
No description provided.