Skip to content

Conversation

@Sleepful
Copy link
Contributor

@Sleepful Sleepful commented Oct 25, 2022

--stdin-filename is necessary for HEEX and should not make a difference for other Elixir files, ref:
https://hexdocs.pm/mix/1.14.1/Mix.Tasks.Format.html
Requires at least Elixir 1.14.0 and for HEEX templates the correct .formatter.exs configuration: https://hexdocs.pm/phoenix_live_view/Phoenix.LiveView.HTMLFormatter.html#module-setup

goes along: lassik/emacs-language-id#23

@lassik
Copy link
Owner

lassik commented Nov 18, 2022

@Sleepful Do you have time to make the above changes?

@Sleepful
Copy link
Contributor Author

@lassik which changes are you referring to?

@Sleepful
Copy link
Contributor Author

Sleepful commented Nov 20, 2022

you might have forgotten to click submit review/request changes

@lassik
Copy link
Owner

lassik commented Nov 20, 2022

you might have forgotten to click submit review/request changes

Sorry! Indeed, it seems I did.

@Sleepful Sleepful requested a review from lassik November 29, 2022 04:56
@lassik lassik merged commit ffbba59 into lassik:master Dec 1, 2022
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