build error about applicationinsights-sdk-labs HOT 4 CLOSED

Sounds like the build server is somehow ending up using an older version of StyleCop? Building locally on Release, with all StyleCop rules running, and I'm not seeing these errors. Do you have a build log you can share so I can check?

(Those errors all seem to be related to using newer C# constructs)

from applicationinsights-sdk-labs.

Do you have access to this: https://mseng.visualstudio.com/AppInsights/ChuckNorris/_build/index?buildId=3743486&_a=summary ?

from applicationinsights-sdk-labs.

Well, this is unexpected.

I squashed my commits before pushing and sending the PR. My local changes compile cleanly, while the pushed changes (and thus the result after the merge) don't. Will try to resolve and send a new PR.

Update: Well, no, turns out StyleCop just generates totally different output when running on VS2017 and VS2015. Even merged code builds clean on a machine with VS2017, but fails on VS2015.

from applicationinsights-sdk-labs.

Update 2: Finally figured what's going on.

The problem only happens when building dirs.proj and not when building WCF.sln, because it builds DependencyCallstacks.sln first. That project references an older version of the StyleCop.MSBuild package, and it seems MSBuild is using that same one for building the WCF.sln solution, despite it referencing a newer version of the package.

PR updating the package on DependencyCallstacks coming up...

from applicationinsights-sdk-labs.