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

Big internals refactoring #761

Open
wants to merge 47 commits into
base: main
Choose a base branch
from
Open

Big internals refactoring #761

wants to merge 47 commits into from

Conversation

avh4
Copy link
Owner

@avh4 avh4 commented Jan 13, 2022

NOTE: If anyone's looking to contribute a PR, you should probably stay up-to-date with this branch, rather than current main.

This is some long-overdue refactoring:

  • Clean up how the main code for the formatter is organized (and start to clearly separate Elm-specific logic from generic text layout logic)
  • Separate code transformations (like removing unnecessary parens, fixing some invalid syntax, etc) from the formatting code. It will now be a separate pass that will be performed first.

@avh4 avh4 self-assigned this Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant