Skip to content

GH2196: Add repository element to nuspec file in NuGetPack tool#2167

Merged
devlead merged 1 commit intocake-build:developfrom
rh072005:feature-AddRepositoryElementToNuGetPack
Jun 21, 2018
Merged

GH2196: Add repository element to nuspec file in NuGetPack tool#2167
devlead merged 1 commit intocake-build:developfrom
rh072005:feature-AddRepositoryElementToNuGetPack

Conversation

@rh072005
Copy link
Copy Markdown
Contributor

No description provided.

@rh072005
Copy link
Copy Markdown
Contributor Author

Having read more about SourceLink this may not actually be needed. If it's useful that's great but if not feel free to close

@devlead
Copy link
Copy Markdown
Member

devlead commented Jun 20, 2018

Is there's an issue to go long with this PR? (as required per Contributing guidelines)

@rh072005
Copy link
Copy Markdown
Contributor Author

Apologies I'd missed that part of the contrib guide. I'll put a note in Gitter now to see if this is something people are interested in and then get an issue raised if so.

@patriksvensson
Copy link
Copy Markdown
Member

@rh072005 The reason for an issue is not (always) to get buy-in, but because we need an issue for our release process. I see no issue with adding this functionality so if you create an issue for this functionality we will merge it after a review 👍

/// <summary>
/// Specifies the package's source code location, allowing IDEs to download and debug the code.
/// </summary>
public class NugetRepository
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Shouldn't this be NuGetRepository instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good spot, I'll get it updated

@rh072005
Copy link
Copy Markdown
Contributor Author

Cheers @patriksvensson , that makes perfect sense. I've raised it as #2196

@devlead devlead changed the title Add repository element to nuspec file in NuGetPack tool GH2196: Add repository element to nuspec file in NuGetPack tool Jun 21, 2018
@devlead devlead force-pushed the feature-AddRepositoryElementToNuGetPack branch from 6041579 to a2ed04d Compare June 21, 2018 22:12
Copy link
Copy Markdown
Member

@devlead devlead left a comment

Choose a reason for hiding this comment

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

LGMT 👍

@devlead devlead merged commit a2ed04d into cake-build:develop Jun 21, 2018
devlead added a commit that referenced this pull request Jun 21, 2018
…develop

* rh072005-feature-AddRepositoryElementToNuGetPack:
  Add repository metadata element to nuget pack * Fixes #2167
@devlead
Copy link
Copy Markdown
Member

devlead commented Jun 21, 2018

@rh072005 your changes have been merged, thanks for your contribution 👍

devlead added a commit that referenced this pull request Jul 6, 2018
* release/0.29.0:
  (build) Updated version and release notes.
  (GH-2094) Allow no dependecy exec of single task * fixes #2094
  Change the legacy pattern validation. Do not rely on Uri.TryCreate, use a regex instead. This will fix the absolute path handling issues. Add tests for testing legacy and new patterns with relative and absolute pathes on all platform.
  (GH-2193) Added new maintainers
  (GH-2203) Added Octopus Deploy release promotion * fixes #2203
  (GH-2095) NUnit Skip non test assemblies * Add --skipnontestassemblies funcionality to CAKE's NUnit3Settings as it exists in original nunit3 test runner * Fixes #2095
  Initial implementation of PushTestResults and PublishCodeCoverage
  Add repository metadata element to nuget pack * Fixes #2167
  Made SelfContained a nullable boolean since no setting means it takes the .Net default of true when runtime is included. Added the logic to include false for self contain and test Fixes #2140
  try resolve vstest.console.exe before guessing it
  Try to find vswhere.exe on the system if the tool is not registered.
  changed the recommended tool for the WiX package in the documentation
  (build) Updated Cake tool to version 0.28.1
  (GH-2188) Corrected link to CLA
  (build) Updated Cake tool to version 0.28.0
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.

4 participants