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

Fixed a typo in TaskItemComparer #240

Merged
merged 1 commit into from
Oct 1, 2015
Merged

Conversation

LordJZ
Copy link
Contributor

@LordJZ LordJZ commented Sep 30, 2015

(Courtesy of Viva64 and their PVS-Studio static analyzer. http://habrahabr.ru/company/pvs-studio/blog/268001/ )

@rainersigwald
Copy link
Member

Looks like this is #81. Can you add that to your commit description?

Related: #186, which has an identical fix but an unclear commit message. I can live without the unit test I asked for over there; clearly we should fix this.

Fixed dotnet#81

(Courtesy of Viva64 and their PVS-Studio static analyzer. http://habrahabr.ru/company/pvs-studio/blog/268001/ )
@LordJZ
Copy link
Contributor Author

LordJZ commented Oct 1, 2015

Cannot build and add tests because of #224, sorry. Amended the commit message.

@AndyGerlicher
Copy link
Contributor

Thanks for updating. Looks good to me as well, this really needs to be fixed.

@rainersigwald
Copy link
Member

LGTM. And hopefully we've figured out #224--it looks like you need to add the VS feature that includes the Windows SDK to get our NuGet stuff to work.

AndyGerlicher added a commit that referenced this pull request Oct 1, 2015
Fixed a typo in TaskItemComparer
@AndyGerlicher AndyGerlicher merged commit 529609f into dotnet:master Oct 1, 2015
@AndyGerlicher AndyGerlicher mentioned this pull request Oct 1, 2015
@LordJZ LordJZ deleted the patch-1 branch October 2, 2015 02:10
radical pushed a commit to radical/msbuild that referenced this pull request Jan 23, 2020
…0121.4 (dotnet#240)

- Microsoft.Dotnet.Toolset.Internal - 3.1.200-preview.20071.4
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.

None yet

4 participants