My concern is that will delay build. We should think about how to exclude analyzers when they are not needed, for example, when we need to quickly recompile a project locally.

I think the analyzer is not critical for us today and we could add it as optional. Optional GitHub check?

PowerShell-CI-windows (Build for Windows Windows Build) #PR-14336-20201206.01 Add DocumentationAnalyzers to build completed in 14m 10s.

My concern is that will delay build. We should think about how to exclude analyzers when they are not needed, for example, when we need to quickly recompile a project locally.

I think the analyzer is not critical for us today and we could add it as optional. Optional GitHub check?

For a fast local build: dotnet build -p:RunAnalyzers=false

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

My concern is that will delay build. We should think about how to exclude analyzers when they are not needed, for example, when we need to quickly recompile a project locally.
I think the analyzer is not critical for us today and we could add it as optional. Optional GitHub check?

For a fast local build: dotnet build -p:RunAnalyzers=false

@iSazonov Perhaps we could add -SkipAnalyzers switch parameter to Start-PSBuild?

@iSazonov Perhaps we could add -SkipAnalyzers switch parameter to Start-PSBuild?

Good point.

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

🎉v7.2.0-preview.8 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes