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

Rename constructors and related parser strings of LongListAlign #467

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sorki
Copy link

@sorki sorki commented Nov 7, 2023

The documentation of these doesn't quite reflect the implementation and it seems to cause some confusion.

This patch tries to address it by renaming LongListAlign constructors to better reflect what they do and fix their documentation in stylish-haskell.yaml.

There is no change in functionality but it introduces a breaking change for users using the Haskell interface. It also adds a (lousy) concept of config option deprecation in form of documenting the deprecated variants in favor of new names.

Related to #462

The documentation of these doesn't quite reflect the implementation
and it seems to cause some confusion.

This patch tries to address it by renaming `LongListAlign`
constructors to better reflect what they do and fix their
documentation in `stylish-haskell.yaml`.

There is no change in functionality but it introduces a breaking
change for users using the Haskell interface. It also adds
a (lousy) concept of config option deprecation in form of documenting
the deprecated variants in favor of new names.

Related to haskell#462
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.

1 participant