-
Notifications
You must be signed in to change notification settings - Fork 98
Open
Description
Summary of the new feature / enhancement
The project does not seem to use any static code analysis.
Adding this would increase code quality and maintainability.
Other Microsoft PowerShell projects has it, like PowerShell itself.
Proposed technical implementation details (optional)
When coding
Add some config to src/code/Microsoft.PowerShell.PSResourceGet.csproj
. I tested by adding this:
<PropertyGroup>
<EnforceCodeStyleInBuild>false</EnforceCodeStyleInBuild>
<TreatWarningsAsErrors>false</TreatWarningsAsErrors>
<AnalysisLevel>8</AnalysisLevel>
<AnalysisMode>Minimum</AnalysisMode>
</PropertyGroup>
For PRs
Analyse on PRs, must have no errors before merge. Maybe by <EnforceCodeStyleInBuild>
?