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.

Sign up for GitHub

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

Jump to bottom

Formatting options #31

Closed
nosami opened this issue Dec 18, 2014 · 5 comments · Fixed by #1526
Closed

Formatting options #31

nosami opened this issue Dec 18, 2014 · 5 comments · Fixed by #1526
Assignees
@Mpdreamz

Comments

@nosami
Copy link
Contributor

nosami commented Dec 18, 2014

We need to serialize these http://source.roslyn.codeplex.com/#Microsoft.CodeAnalysis.CSharp.Workspaces/Formatting/CSharpFormattingOptions.cs,dac739b7b8a53984

and http://source.roslyn.codeplex.com/#Microsoft.CodeAnalysis.Workspaces/Formatting/FormattingOptions.cs,2ce68197c6bbf674
into the config file, and load them on solution start into Workspace.Options

We also need a web UI for managing these.
@nosami nosami mentioned this issue Dec 18, 2014
Closed
@jrieken
Copy link
Contributor

jrieken commented Dec 18, 2014

config dump in here: https://github.com/OmniSharp/omnisharp-roslyn/tree/joh/codeformat

@davidfowl
Copy link
Member

We should look at editor config too

@nosami
Copy link
Contributor Author

nosami commented Dec 18, 2014

@Mpdreamz said he'd look at EditorConfig.

@Yantr.io has offered to help with web UI for configuration.

@Mpdreamz Mpdreamz self-assigned this Feb 6, 2015
@Mpdreamz
Copy link
Member

Mpdreamz commented Feb 6, 2015

Assigning this to me since I'm taking a stab at plugging editorconfig in. Finally making use of the rewrite to managed code :D https://www.nuget.org/packages/editorconfig

@jrieken
Copy link
Contributor

jrieken commented Feb 9, 2015

👍

@Mpdreamz Mpdreamz mentioned this issue Feb 13, 2015
Merged
@bstockus bstockus mentioned this issue Oct 2, 2016
Merged
@filipw filipw mentioned this issue Jun 15, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mpdreamz Mpdreamz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added support for .editorconfig filipw/omnisharp-roslyn
4 participants
@davidfowl @Mpdreamz @nosami @jrieken