Skip to content
This repository was archived by the owner on Dec 5, 2022. It is now read-only.

Add update to show support for vs 2019 #221

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions docs/contribute.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,9 @@ Clone the repository to a local directory. Rest of this article assumes
```

If you're planning to use **Visual Studio** as development environment, please
install `VS 2017 RC 2` and the `.NET Core and Docker (Preview)` workload installed. See download
link [here](https://www.microsoft.com/net/core#windowsvs2017).
install `VS 2019` and the `.NET Core and Docker (Preview)` workload installed. See download
link [here](https://www.microsoft.com/net/core#windowsvs2017).
Copy link
Contributor

Choose a reason for hiding this comment

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

@vritant24 : It looks like this download link is still pointing to vs2017. Can we update that please?

Ensure that .NET Portable library targeting pack is installed with the workloads.

If you're _not_ planning to use **Visual Studio** and only use CLI. You will need to install [.Net 46 targeting pack](https://www.microsoft.com/en-us/download/details.aspx?id=48136). The download link has two msis. Both needs to be installed. Otherwise build will fail asking to install net 46.

Expand Down