豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

Fix broken link to Windows 10 SDK#3697

Merged
zivkan merged 2 commits intoNuGet:devfrom
bjorkstromm:patch-1
Oct 14, 2020
Merged

Fix broken link to Windows 10 SDK#3697
zivkan merged 2 commits intoNuGet:devfrom
bjorkstromm:patch-1

Conversation

@bjorkstromm
Copy link
Copy Markdown
Contributor

@bjorkstromm bjorkstromm commented Sep 30, 2020

Bug

Fixes: NuGet/Home#10099
Regression: No

  • Last working version: -
  • How are we preventing it in future: -

Fix

Details: This fixes a broken link in contributing documentation.

Testing/Validation

Tests Added: No
Reason for not adding tests: Only documentation
Validation: -

Copy link
Copy Markdown
Member

@nkolev92 nkolev92 left a comment

Choose a reason for hiding this comment

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

Thanks @mholo65

Comment thread CONTRIBUTING.md Outdated
- Visual Studio extension development.
- Desktop development with C++
- [Windows 10 SDK](https://dev.windows.com/en-US/downloads/windows-10-sdk)
- [Windows 10 SDK](https://developer.microsoft.com/en-us/windows/downloads/windows-10-sdk/)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not if this is still needed for building. I think it's needed if you want to run all E2E tests though.

I created a follow up. NuGet/Home#10096

@nkolev92
Copy link
Copy Markdown
Member

ah, I forgot, can you please create a tracking issue for this @mholo65

We want to have every commit correlated to an issue.

@bjorkstromm
Copy link
Copy Markdown
Contributor Author

@nkolev92 thanks! Submitted issue and updated description.

Comment thread CONTRIBUTING.md Outdated
@zivkan zivkan added the Community PRs created by someone not in the NuGet team label Oct 1, 2020
@zivkan
Copy link
Copy Markdown
Member

zivkan commented Oct 3, 2020

@mholo65 This branch also needs to be rebased onto the latest dev.

bjorkstromm and others added 2 commits October 5, 2020 08:48
Co-authored-by: campersau <buchholz.bastian@googlemail.com>
@bjorkstromm
Copy link
Copy Markdown
Contributor Author

@zivkan rebased.

@zivkan
Copy link
Copy Markdown
Member

zivkan commented Oct 14, 2020

This must have been the unluckiest PR we ever had. I swear I had to run about 10-15 builds before there was one without flaky failures. Finally a green build 🥳 . @mholo65 thanks for your contribution.

@zivkan zivkan merged commit 6a64f85 into NuGet:dev Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Community PRs created by someone not in the NuGet team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows SDK 10 link is broken in NuGet.Client Contribution guide

4 participants