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

Spaces after comma in union type? #101

Open
tk3369 opened this issue Sep 4, 2021 · 1 comment
Open

Spaces after comma in union type? #101

tk3369 opened this issue Sep 4, 2021 · 1 comment

Comments

@tk3369
Copy link

tk3369 commented Sep 4, 2021

What's the recommendation for

T = Union{Int, String}

versus

T = Union{Int,String}
@nickrobinson251
Copy link
Contributor

nickrobinson251 commented Oct 7, 2021

we should decide and write something in the main guide about this!

the current status quo is that Blue Style has no preference here... but in JuliaFormatter.jl BlueStyle has to default to one or the other and so far it defaults to no whitespace (#70)

edit: if i remember rightly, the reason for the defaulting to whitespace_in_typedefs = false was that having the whitespace led to a lot of method definitions going past the line length limit and getting changed to be written over more lines. which arguably is fine. but i think it was the reason.

(xref #77 too)

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

No branches or pull requests

2 participants