Skip to content

Add elm-format #100

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

Merged
merged 5 commits into from
Sep 3, 2022
Merged

Add elm-format #100

merged 5 commits into from
Sep 3, 2022

Conversation

elken
Copy link
Contributor

@elken elken commented Jun 3, 2022

Has the change to not report errors if it fails, I experiemented with using the below shell script which can be used instead if desired, though it wasn't 100% accurate.

elm-format --yes "$1" || elm make "$1"

@elken
Copy link
Contributor Author

elken commented Jun 3, 2022

Initial failure was #99

@raxod502
Copy link
Member

Oh, sorry, I think I missed this. Let me take a look.

@elken
Copy link
Contributor Author

elken commented Jul 17, 2022

Aware I can merge my own PRs now, personally unsure about doing that but if you're happy with this I can merge this in too :) @raxod502

@raxod502
Copy link
Member

raxod502 commented Sep 3, 2022

Sorry, should have gotten back to you much earlier about this. Yeah, please feel free.

@raxod502 raxod502 merged commit 4d59a9b into radian-software:main Sep 3, 2022
@elken elken deleted the feat/elm-format branch September 4, 2022 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants