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

Feature: Align brackets #706

Closed
wants to merge 38 commits into from

Conversation

nojaf
Copy link
Contributor

@nojaf nojaf commented Feb 28, 2020

Alternative style of formatting records, arrays and lists.
Would resolve #453
Might need some more usecases and be tested in the wild.

nojaf added 28 commits February 3, 2020 19:21
# Conflicts:
#	src/Fantomas/CodePrinter.fs
#	src/Fantomas/Context.fs
@nojaf nojaf requested a review from jindraivanek February 28, 2020 16:50
docs/Documentation.md Outdated Show resolved Hide resolved
docs/Documentation.md Show resolved Hide resolved
SpaceBeforeSemicolon = true })

[<Test>]
let ``single member record stays on oneline`` () =
Copy link
Contributor

Choose a reason for hiding this comment

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

I think there should be a space between one and line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, thanks.

@nojaf nojaf added the v4 label Mar 16, 2020
@nojaf nojaf removed the request for review from jindraivanek March 16, 2020 17:49
@nojaf
Copy link
Contributor Author

nojaf commented Mar 16, 2020

Still WIP, requires #721 to be merged first.

@nojaf
Copy link
Contributor Author

nojaf commented Apr 14, 2020

Closing in favour of #746

@nojaf nojaf closed this Apr 14, 2020
@nojaf nojaf deleted the feature/bracket-on-separate-line branch December 11, 2020 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: more flexible record whitespace
3 participants