Skip to content

Reword misleading documentation for layoutPretty - #229

Merged
sjakobi merged 1 commit into
haskell-prettyprinter:masterfrom
benjamin-hodgson:patch-1
Jul 9, 2022
Merged

sjakobi merged 1 commit into
haskell-prettyprinter:masterfrom
benjamin-hodgson:patch-1

Conversation

@benjamin-hodgson

Copy link
Copy Markdown
Contributor

The docs implied that layoutPretty looks ahead by a single text element, which is not accurate - it looks ahead to the end of the line. I also reworded the layoutSmart docs, which implied that it tries to fit the whole document into one line.

The docs implied that `layoutPretty` looks ahead by a single text element, which is not accurate - it looks ahead to the end of the line. I also reworded the `layoutSmart` docs, which implied that it tries to fit the whole document into one line.

@sjakobi sjakobi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@sjakobi
sjakobi merged commit 0dbe08f into haskell-prettyprinter:master Jul 9, 2022
@benjamin-hodgson
benjamin-hodgson deleted the patch-1 branch July 23, 2022 13:19
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