Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Format code according to default style #3

Closed
wants to merge 1 commit into from
Closed

Conversation

djc
Copy link
Member

@djc djc commented May 9, 2019

I don't know how attached you are to your particular style, but, FWIW, I found the style sufficiently different from what I am used to that it was a bit distracting (especially your alignment of : characters).

@est31
Copy link
Member

est31 commented May 10, 2019

The : are placed with space before no space after whenever they are followed by a type or type constraint, and placed with space before and after whenever they are used for assignments in struct initializers. The usage is fundamentally different and I'm not really happy about both how the language chose the same syntax for both nor about how the official format guidelines chose the same format for both. Therefore I format the : like I format a = when it's used like one. Also I prefer using tabs :).

Generally I'd prefer to keep it in my personal style that I use for other projects as well.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants