Add static code analysis

Abierto
#1,856 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
csharp
Área
ci-cd, tooling

Línea de trabajo

Comienza con src/code/Microsoft.PowerShell.PSResourceGet.csproj y revisa las propiedades de análisis propuestas y su comportamiento actual durante la compilación. Después, inspecciona las comprobaciones de pull request del repositorio para determinar dónde debería ejecutarse el análisis. Se considera terminado cuando el análisis estático está configurado y los pull requests con errores no se pueden fusionar.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Area-Build&Release feature_request
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>?

Lenguaje dominante
C#
Estrellas
576
Forks
114
Merge medio
23 h 17 min
PR fusionados (30 d)
8

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de PowerShell/PSResourceGet

Todos los issues de PowerShell/PSResourceGet

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.