Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error

Allow Code Analysis option does not apply to Roslyn analyzers
distilled
#1 Posted : Saturday, June 27, 2020 4:29:17 PM(UTC)
Rank: Member

Groups: Registered
Joined: 8/3/2012(UTC)
Posts: 11

Was thanked: 3 time(s) in 3 post(s)
Not sure if this is the right place for this feedback, or if a feature request would be more appropriate.

Roslyn analyzers provided by NuGet packages can significantly increase the time it takes to build a project. NCrunch has a configuration setting to disable code analysis on build. However, this doesn't also disable Roslyn analyzers. It seems like if this setting is configured to disallow code analysis then Roslyn analyzers should not be included and executed when NCrunch builds a project.
Remco
#2 Posted : Monday, June 29, 2020 6:43:54 AM(UTC)
Rank: NCrunch Developer

Groups: Administrators
Joined: 4/16/2011(UTC)
Posts: 6,155

Thanks: 795 times
Was thanked: 1053 time(s) in 1002 post(s)
distilled;14827 wrote:
Not sure if this is the right place for this feedback, or if a feature request would be more appropriate.

Roslyn analyzers provided by NuGet packages can significantly increase the time it takes to build a project. NCrunch has a configuration setting to disable code analysis on build. However, this doesn't also disable Roslyn analyzers. It seems like if this setting is configured to disallow code analysis then Roslyn analyzers should not be included and executed when NCrunch builds a project.


Good catch. Yes, disabling these by default seems sensible and it's also what the configuration setting implies. We'll look into it.


Users browsing this topic
Guest
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

YAF | YAF © 2003-2011, Yet Another Forum.NET
This page was generated in 0.187 seconds.