Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.
/ NuGet.Jobs Public archive

Update jobs to use NuGet.Services.Logging 2.1.1 #127

Closed
wants to merge 1 commit into from

Conversation

dtivel
Copy link
Contributor

@dtivel dtivel commented Apr 5, 2017

@@ -48,8 +48,8 @@
<Reference Include="Newtonsoft.Json, Version=9.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
<HintPath>..\..\packages\Newtonsoft.Json.9.0.1\lib\net45\Newtonsoft.Json.dll</HintPath>
</Reference>
<Reference Include="NuGet.Services.Logging, Version=2.1.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
<HintPath>..\..\packages\NuGet.Services.Logging.2.1.0\lib\net452\NuGet.Services.Logging.dll</HintPath>
<Reference Include="NuGet.Services.Logging, Version=2.1.1.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL">
Copy link
Member

Choose a reason for hiding this comment

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

We talked about two things a while back to release a new (stable) .nupkg:

  1. Quality is verified by the .nupkg's own CI as well as a verification of the consumer on dev. Has this been done for these jobs?
  2. Stable version is pinned in MyGet.

Has these been done?

Copy link
Member

Choose a reason for hiding this comment

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

(previously we had talked about NuGetGallery.Core, and verification with Gallery functional tests, but the analogy here is manually verification of the jobs)

@dtivel dtivel closed this Apr 5, 2017
@dtivel dtivel deleted the dev-dtivel-update-logging branch April 5, 2017 19:18
joelverhagen pushed a commit that referenced this pull request Sep 27, 2019
Merging Dev into Master
joelverhagen pushed a commit that referenced this pull request Oct 26, 2020
Merging Dev into Master
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