-
Notifications
You must be signed in to change notification settings - Fork 146
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
Create a separate package for CommonMark.NET.Console #123
base: master
Are you sure you want to change the base?
Conversation
daa22d3
to
9eb6538
Compare
9eb6538
to
3f924d3
Compare
<projectUrl>https://github.com/Knagis/CommonMark.NET/</projectUrl> | ||
<iconUrl>https://raw.githubusercontent.com/dcurtis/markdown-mark/master/png/32x20.png</iconUrl> | ||
<requireLicenseAcceptance>false</requireLicenseAcceptance> | ||
<summary>Fastest .NET library for converting Markdown documents to HTML.</summary> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 I didn't see a way to preserve the <summary>
element, but I believe I preserved the rest of the metadata.
<file src="CommonMark\bin\$configuration$\portable-net4+sl5+win+wpa+wp8\CommonMark.*" target="lib\portable-net40+sl50+wp80+win+wpa81+MonoAndroid10+MonoTouch10\" /> | ||
<file src="CommonMark\bin\$configuration$\portable-net45+win+wpa+wp8\CommonMark.*" target="lib\portable-net45+wp80+win+wpa81+MonoAndroid10+xamarinios10+MonoTouch10\" /> | ||
<file src="CommonMark\bin\$configuration$\netstandard1.0\CommonMark.*" target="lib\netstandard1.0\" /> | ||
<file src="CommonMark.Console\bin\$configuration$\net45\CommonMark.Console.exe" target="tools\cmark.exe" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 The tool was moved to a separate NuGet package.
@@ -0,0 +1,18 @@ | |||
{ | |||
"$schema": "https://raw.githubusercontent.com/AArnott/Nerdbank.GitVersioning/master/src/NerdBank.GitVersioning/version.schema.json", | |||
"version": "0.15.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝 This is capable of calculating unique version numbers for each commit on build, but for now I've hard-coded all four digits of the version number to match the behavior prior to this pull request. The partial adoption of Nerdbank.GitVersioning causes the ThisAssembly
class to be generated during the build, which is used by InitializeVersion
to provide version information now that the nuspec file is generated after the assembly has already been built.
The behavior change introduced by this involves the [AssemblyVersion]
and [AssemblyFileVersion]
attributes, which now read 0.15.0.0
and 0.15.1.0
, respectively. Prior to this change, the NuGet package version was 0.15.1.0, but both of the attributes were 0.1.0.0.
Please see the notes in review and feel free to ask any questions before deciding to merge.