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

--noSpaceBeforeColon doesn't work #499

Closed
FraserWaters-GR opened this issue Oct 4, 2019 · 1 comment
Closed

--noSpaceBeforeColon doesn't work #499

FraserWaters-GR opened this issue Oct 4, 2019 · 1 comment

Comments

@FraserWaters-GR
Copy link

Description

--noSpaceBeforeColon should toggle the FormatConfig.SpaceBeforeColon option, but because SpaceBeforeColon defaults to false, and the --no flags can only toggle an option false SpaceBeforeColon is always false.

SpaceBeforeColon should either default to true (like the other SpaceBefore options) or the cli parser should be extended to allow --noSpaceBeforeColon=true/false.

Also documentation.md has a list of the default values of these options but it doesn't match the code in FormatConfig.fs.

@nojaf
Copy link
Contributor

nojaf commented Oct 7, 2019

Ah yes, documentation might be out of sync with some new defaults and cli should be updated as well.

@nojaf nojaf closed this as completed in 8fbdbe3 Oct 11, 2019
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