Brian Collamore

.Net Core environment with --no-restore not working

2

Per https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-build: "Running 'dotnet build' is equivalent to running 'dotnet msbuild -restore'" In order to not restore in a net core environment, we need to pass --no-restore. Therefore, setting Restore...

Superlinter does not find actionlint.yaml in the default location of .github/linters.

3

## Describe the bug Superlinter does not find actionlint.yaml in the default location of .github/linters. (It will find actionlint.yml.) ## Expected behavior A path of .github/linters/actionlint.yaml should work by default...

Starting MsbuildAnalyzer

4

This is not worthy of serious consideration, as it is not operational yet. But it gives a flavor for how this could be accomplished. I only present what is available...

dev(buildalyzer): Begin encapsulating Buildalyzer

5

Start encapsulating Buildalyzer, so as to begin exploring possible alternatives in the future. * New interfaces/classes hierarchy to encapsulate Buildalyzer classes * ProjectOrchestrator is updated - it's an entry point...

Avoid method calls as parameter for a methods

1

Investigate and apply 3rd-party popular analyzer, or create. _Originally posted by @ynsehoornenborg in https://github.com/philips-software/roslyn-analyzers/pull/159#discussion_r1045449283_

Avoid multiple conditions in same line for readability

1

Investigate 3rd party analyzer or create. Consider putting each condition on its own line for readability _Originally posted by @ynsehoornenborg in https://github.com/philips-software/roslyn-analyzers/pull/159#discussion_r1045453437_

I believe I've seen this in a popular 3rd party analyzer package already somewhere. Consider enabling it on our project, and/or creating the analyzer if unavailable elsewhere. _Originally posted by...

Every Linq statement on a separate line for readability

2

Is there a popular 3rd party analyzer package to auto-enforce this everywhere? Or, create? _Originally posted by @ynsehoornenborg in https://github.com/philips-software/roslyn-analyzers/pull/159#discussion_r1045451380_

Exclude this analyzer from generated code