-
Notifications
You must be signed in to change notification settings - Fork 75
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
Future of this project with regards to VS 2017 #49
Comments
Unless someone wants to and sees a reason. I personally have no plans to bring this extension over to 2017.
and
From what i gathered many more source formatting options will be expressible as editor config extension properties so i'm super hopeful this will receive continued love from Microsoft! ❤️ If someone feels these are important enough limitations to warrant a release for 2017 please let me or other members of the organization know, we happily accept PR's and add to the contributor list for the repos if needed! |
I think it is great the VS2017 has builtin EditorConfig support. The release notes are about the RC. Those limitations you mention may even be fixed in the RTM. |
Yup some of the editorconfig extension properties that are going into roslyn just got pulled in a vs2017-rc2 branch: dotnet/roslyn#15029 Good times ahead! |
FWIW i added a note on VS2017 and Resharper to the readme https://github.com/editorconfig/editorconfig-visualstudio |
I'm not sure if I am correct, but isn't the support in VS2017RC only for C#? |
Although I haven't tested the RC, I assume it is a Visual Studio feature, not a language specific one. So it should work for all languages that VS supports. |
Ah sorry. It works with C++, the problem was, that it doesn't support solution wide .editorconfig. I have to put it inside of every project. |
As said, haven't seen it in action yet but I assume when you put the .editorconfig file in the right place (the root directory of your solution) it will work as expected. Don't see why they would implement it differently. Do you have any reference that a solution wide .editorconfig file doesn't work? Did you see this behavior yourself? Is the "root = true" set in the right place? |
So I tried it myself first. Putting .editorconfig file into the solution folder does nothing, while putting it into the project folder results in correct indentation setting. The file is just simple:
There's also this topic in the support page, but it's closed as not a bug. |
thanks for reporting @spades789 👍 I've pinged the folks over at MS (over email) that I know worked on the EditorConfig integration to continue the conversation. It would be a real shame if the native editorconfig does not resolve files according to the spec. |
@spades789 MS let me know that it should work and now that I have VS2017 installed myself I can confirm it works as expected on my end. Also checkout: https://github.com/madskristensen/EditorConfigLanguage for additional editorconfig integrations (the inheritance visualizer could maybe help debug your problem). |
I was just going over the release notes of VS 2017 and noticed this:
I suppose that this extension will be deprecated then in favour of the built-in support? Either way, thanks for all the work so far, since it probably encouraged Microsoft to add built-in support :)
The text was updated successfully, but these errors were encountered: