Skip to content

Upgrade to VS 2017#179

Merged
natemcmaster merged 1 commit into
devfrom
namc/vs2017
Dec 15, 2016
Merged

Upgrade to VS 2017#179
natemcmaster merged 1 commit into
devfrom
namc/vs2017

Conversation

@natemcmaster

Copy link
Copy Markdown

Upgrade this solution to build with VS 2017 and dotnet-cli 1.0.0-preview4.

cc @Eilon @muratg

Comment thread build/common.props
<PropertyGroup>
<RepositoryUrl>https://github.com/aspnet/PlatformAbstractions</RepositoryUrl>
<RepositoryType>git</RepositoryType>
<AssemblyOriginatorKeyFile>$(MSBuildThisFileDirectory)Key.snk</AssemblyOriginatorKeyFile>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

MSBuildThisFileDirectory value has a trailing backspace?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@natemcmaster

Copy link
Copy Markdown
Author

⌚ on aspnet/Universe#466 to merge


<PropertyGroup>
<Description>Command-line parsing API. Commonly used types:
Microsoft.Extensions.CommandLineUtils.CommandLineApplication

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Love the newlines! 😄 Do they end up showing up correctly in the package's nuspec?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yup!

image

Comment thread build/common.props
<Import Project="..\.build\common.props" Condition="Exists('..\.build\common.props')" />

<PropertyGroup>
<RepositoryUrl>https://github.com/aspnet/PlatformAbstractions</RepositoryUrl>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

https://github.com/aspnet/Common

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yeah, I caught that later :)

27b4795#diff-9780bf7a72094f1ac4b023e438b548ba

natemcmaster pushed a commit that referenced this pull request Nov 5, 2018
@ghost ghost locked as resolved and limited conversation to collaborators May 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants