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

default_juliaformatter_config not same as JuliaFormatter default? #1256

Open
xgdgsc opened this issue Oct 31, 2023 · 3 comments
Open

default_juliaformatter_config not same as JuliaFormatter default? #1256

xgdgsc opened this issue Oct 31, 2023 · 3 comments

Comments

@xgdgsc
Copy link
Contributor

xgdgsc commented Oct 31, 2023

function default_juliaformatter_config(params)
from my understanding it is different than JuliaFormatter.jl default . Is it the reason I have to create a .JuliaFormatter.toml file for every project? Especially the margin setting.

@davidanthoff
Copy link
Member

Yes, the defaults are different. In general the defaults in VS Code are meant to be very minimal, i.e. a lot less opiniated than the current defaults for JuliaFormatter. We at some point had a plan how to unify this, but we somehow got stuck in the middle... Probably worth revisiting this, as it is pretty annoying...

@xgdgsc
Copy link
Contributor Author

xgdgsc commented Nov 1, 2023

Other options might not make a huge difference in view. Would you mind me changing margin to 92 before a unified solution is available ?

I' m surprised when I heard some one use https://marketplace.visualstudio.com/items?itemName=0h7z.vscode-julia-format&ssr=false#overview for formatting.

@davidanthoff
Copy link
Member

Would you mind me changing margin to 92 before a unified solution is available?

I think we actually agreed that JuliaFormatter would change its default to be less opinionated. One key thing is that by default we don't want any line breaks added or removed, so I don't think we should change the VS Code default right now, just to change it back again once we unify.

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