Skip to content

Conversation

simenhagelid
Copy link
Contributor

Added floating version for PackageReference Microsoft.Extensions.Logging to 2.*

@nblumhardt nblumhardt changed the base branch from master to dev April 23, 2019 00:59
@nblumhardt nblumhardt changed the title Master Use a version wildcard for MEL dependency Apr 23, 2019
@nblumhardt
Copy link
Member

Thanks! Let's give this a try 👍

I've re-targeted it to dev - will squash and merge to try cleaning up the history a bit :-)

@nblumhardt nblumhardt merged commit 5a8be58 into serilog:dev Apr 23, 2019
@nblumhardt nblumhardt mentioned this pull request Apr 23, 2019

<ItemGroup Condition=" '$(TargetFramework)' == 'net461' or '$(TargetFramework)' == 'netstandard2.0' ">
<PackageReference Include="Microsoft.Extensions.Logging" Version="2.0.0" />
<PackageReference Include="Microsoft.Extensions.Logging" Version="2.*" />
Copy link
Contributor

Choose a reason for hiding this comment

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

note that this wont effect the resulting package. on each build the resultant nuget will have the current release version of Microsoft.Extensions.Logging hard coded as a package dependency

Copy link
Member

Choose a reason for hiding this comment

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

Argh, thanks Simon - lost track of this particular footgun :-)

Copy link
Contributor

Choose a reason for hiding this comment

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

I only remember due to my missing foot

Copy link
Member

Choose a reason for hiding this comment

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

😂

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