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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add idris-format module with idris-format-align-lhs command #615

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

keram
Copy link
Contributor

@keram keram commented Feb 2, 2023

Why:
Sometimes it makes nicer to align left hand side to columns.

Demo:

output-2023-02-02-19:16:59

output-2023-02-02-19:19:45

This is first version and as next step it could be nice generalise the solution for aligning different types left hands sides.
Also not sure if this should live in idris-mode or rather as separate package 馃

@jfdm
Copy link
Contributor

jfdm commented Feb 8, 2023

...live in idris-mode or rather as separate package...

This is more an open discussion rather than a strict 'we must do'...

@keram I have been having a think about idris-format, specifically if this is something that is idris specific or something that is applicable to more languages.

One could even argue, and perhaps a more long term solution, is for code formatting to be part of the Idris2 compiler's operation and communicated through the IDE-Protocol.

More so updating Idris1 is not a priority, and having the idris-format package available until Idris1 is truly deprecated would be beneficial.

So perhaps these points are something to bear in mind.

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.

None yet

2 participants