Build a Nupkg without a .nuspec

[ErikEJ]

fixes #283

@jmezach It looks like the Nuspec properties /metadata (like PackageProject) are not passed? Any suggestions for how to fix it?

[ErikEJ]

@jmezach All is good, I was looking in the wrong place 😄

This is an example of the auto generated .nuspec from the test project:

<?xml version="1.0" encoding="utf-8"?>
<package xmlns="http://schemas.microsoft.com/packaging/2012/06/nuspec.xsd">
  <metadata>
    <id>TestProject</id>
    <version>2.8.0-beta.29.g04d634525c</version>
    <authors>MSBuild.SDK.SqlProj team</authors>
    <projectUrl>https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/</projectUrl>
    <description>Package Description</description>
    <dependencies>
      <group targetFramework=".NETStandard2.0" />
    </dependencies>
  </metadata>
  <files>
    <file src="C:\Code\Github\MSBuild.Sdk.SqlProj\test\TestProject\bin\Debug\netstandard2.0\TestProject.dacpac" target="tools\TestProject.dacpac" />
  </files>
</package>

[jmezach]

LGTM

[ErikEJ]

@jmezach Curious how this worked before? https://github.com/rr-wfm/MSBuild.Sdk.SqlProj/blob/master/src/MSBuild.Sdk.SqlProj/Sdk/Sdk.targets#L67

[ErikEJ]

Or maybe it never worked?

.nuspec format:

<packageTypes>
      <packageType name="Dacpac" />
 </packageTypes>

[jmezach]

@ErikEJ I'm guessing that setting that packageType property didn't actually do anything, so it wouldn't show up in the resulting .nuspec anyway so that's probably why it just worked. We should probably remove it though to avoid confusion in the future.

[ErikEJ]

@jmezach I removed it! 😄 - certainly confused me!

[ErikEJ]

@jmezach I just noticed this: https://github.com/rr-wfm/MSBuild.Sdk.SqlProj?tab=readme-ov-file#packaging-standalone-dacpacs - should the readme also have it removed in this section?

[jmezach]

Yeah, I'd say so