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

Code Formatting #17

Closed
svermeulen opened this issue May 3, 2013 · 8 comments
Closed

Code Formatting #17

svermeulen opened this issue May 3, 2013 · 8 comments

Comments

@svermeulen
Copy link
Contributor

Is there plans to add a hook to trigger NRefactory code formatting? I saw no mention of it in the TODO list so I thought I'd mention it.

@nosami
Copy link
Contributor

nosami commented May 3, 2013

It's definitely on my list of stuff to do. Should be pretty straightforward.

I added the C# indent file yesterday that helps somewhat with Vim's built in = indenting... but it doesn't have all the capabilities of NRefactory.

@chtenb
Copy link
Contributor

chtenb commented May 3, 2013

Would Omnisharp provide significant better formatting than astyle does?

@nosami
Copy link
Contributor

nosami commented May 3, 2013

I'm not sure that it would.... but it seems to me that seeing as we already have NRefactory installed, we may as well use it. It isn't much code for me to add to OmniSharp and it removes the astyle dependency.
It has a ton of options for code formatting (see https://github.com/icsharpcode/NRefactory/blob/master/ICSharpCode.NRefactory.CSharp/Formatter/CSharpFormattingOptions.cs)

@chtenb
Copy link
Contributor

chtenb commented May 3, 2013

Okay, sounds reasonable. When its implemented I may consider integration into vim-autoformat. (https://github.com/Chiel92/vim-autoformat)

@nosami
Copy link
Contributor

nosami commented May 3, 2013

I added a code formatter command. It's currently hard coded to format the same way that VS does it. Let me know if this will do for now. Probably no time to spend on this until next week.

@svermeulen
Copy link
Contributor Author

Great! I will try it out

@svermeulen
Copy link
Contributor Author

I can confirm that this works quite well for my purposes.

@nosami
Copy link
Contributor

nosami commented Jun 29, 2013

Closing for now, it does what I need.

@nosami nosami closed this as completed Jun 29, 2013
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

3 participants