Skip to content
This repository has been archived by the owner on Jan 24, 2021. It is now read-only.

Remove nuspecs as no longer needed #2658

Merged
merged 1 commit into from
Dec 23, 2016

Conversation

jchannon
Copy link
Member

@jchannon jchannon commented Dec 22, 2016

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the Nancy code style guidelines
  • I have provided test coverage for my change (where applicable)

Description

We use project.json now so *nuspecs are no longer required

@jvandertil
Copy link
Contributor

Seems like a good change, I'm assuming this is also the case with the new csproj format?

@thecodejunkie
Copy link
Member

@jvandertil Our build scripts use dotnet pack to build all NuGet packages these days, so the *.nuspec files aren't really used anymore. Once Visual Studio 2017 gets closer to RTM then we will have a look at how we will support the new csproj-format. It's not all black and white, I am afraid, since from what I have gathered they will not back port the tooling to Visual Studio 2015 and we will have to assume that for a while that's what most of our users will continue to support. Of course we want to move over to the new csproj format so with a bit of luck we can support both scenarios without too much hassle (supporting csproj and project.json today was a major PITA because of the tooling support)

@thecodejunkie thecodejunkie merged commit 376c4f7 into NancyFx:master Dec 23, 2016
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.

4 participants