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

Update documentation to recommend VS code 2019 #524

Closed
wants to merge 1 commit into from

Conversation

ethanchewy
Copy link
Contributor

VS code versions > 2019 doesn't yet support important libraries that are needed to debug/modify runner code effectively such as .NET, etc.

Ex: dotnet/vscode-csharp#2507

VS code versions > 2019 doesn't yet support important libraries that are needed to run runner such as .NET, etc. 

Ex: dotnet/vscode-csharp#2507

[Using Visual Studio](https://code.visualstudio.com/docs)
We recommend using [Visual Studio 2019](https://visualstudio.microsoft.com/vs/) since it has built in support for .NET and other libraries that are helpful for contributing and running this repo while newer versions of Visual Studio does not have support for these libraries as of yet.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would expect the older VS version might not work with dotnet core, not the newer version.

What error do you get?

I would not say We recommend using [Visual Studio 2019] since VsCode is a better IDE for this repo since the runner is for Windows/Linux/macOS, but you can only use VS in Windows.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a lot of issues with omnisharp like: dotnet/vscode-csharp#2507. But when I reinstall Visual Studio Code, the error disappeared and I wasn't able to replicate it so I'm not sure what's going on. I'll close this PR for now :P

@ethanchewy ethanchewy closed this Jun 8, 2020
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

Successfully merging this pull request may close these issues.

2 participants