serilog-contrib/Serilog.Sinks.Logz.Io

Unoptimized assembly detected: 'Serilog.Sinks.Logz.Io.dll'

Closed this issue · 3 comments

While upgrading NuGet Trends's package reference of Serilog.Sinks.Logz.Io from version 2.2.2 to 3.0.1 UnoptimizedAssemblyDetector alerted the DLL wasn't compiled with optimizations:

/Users/bruno/.nuget/packages/unoptimizedassemblydetector/0.1.1/build/UnoptimizedAssemblyDetector.targets(14,5): warning : Unoptimized assembly detected: 'Serilog.Sinks.Logz.Io.dll' at /Users/bruno/.nuget/packages/serilog.sinks.logz.io/3.0.1/lib/netstandard2.1/Serilog.Sinks.Logz.Io.dll [/Users/bruno/git/nuget-trends/src/NuGetTrends.Scheduler/NuGetTrends.Scheduler.csproj]
/Users/bruno/.nuget/packages/unoptimizedassemblydetector/0.1.1/build/UnoptimizedAssemblyDetector.targets(14,5): warning : Unoptimized assembly detected: 'Serilog.Sinks.Logz.Io.dll' at /Users/bruno/.nuget/packages/serilog.sinks.logz.io/3.0.1/lib/netstandard2.1/Serilog.Sinks.Logz.Io.dll [/Users/bruno/git/nuget-trends/src/NuGetTrends.Web/NuGetTrends.Web.csproj]
    2 Warning(s)

I don't see an automation in the repo to review, but is it possible packaging was done without -c Release? Shameless plugging a blog post on this exact situation that might give some context.

Update: I checked and 3.0.0 was compiled with optimizations.

Seems like version 5.1.0 is still not optimized
/Users/bruno/.nuget/packages/unoptimizedassemblydetector/0.1.1/build/UnoptimizedAssemblyDetector.targets(14,5): warning : Unoptimized assembly detected: 'Serilog.Sinks.Logz.Io.dll' at /Users/bruno/.nuget/packages/serilog.sinks.logz.io/5.1.0/lib/net5.0/Serilog.Sinks.Logz.Io.dll [/Users/bruno/git/nuget-trends/src/NuGetTrends.Scheduler/NuGetTrends.Scheduler.csproj]

Sorry about that, should improve my release process a bit :)
Latest one (6.0.0) should be optimized again.