PowerShell / PowerShell/PSScriptAnalyzer

Update/standardize the internal custom `VariableAnalysis` classes

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

Nadie ha tomado este issue todavía.

Area - Engine Consider - 2.0 Issue - Enhancement
Lenguaje dominante
C#
Estrellas
2.2k
Forks
414
Merge medio
13 h 1 min
PR fusionados (30 d)
2

Descripción

This addresses technical debt.

... the VariableAnalysis classes that were forked from PowerShell's own classes 4 years ago. Those classes were not updated since and have small bugs that leads to certain types of analysis not being possible. Unfortunately, the original maintainers have taken only a subset of it and added a lot of custom code to it. I once tried to just delete it and replace it with the code from PowerShell and some of the challenges were that SMA are very entangled with each other in terms of dependencies and it compiles against .Net Core, but PSSA compiles against .Net Standard.

(https://github.com/PowerShell/PSScriptAnalyzer/pull/1277#issuecomment-508929230)

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.

Línea de trabajo

Comienza leyendo la discusión enlazada en PowerShell/PSScriptAnalyzer#1277 e identificando las clases internas de VariableAnalysis y sus dependencias. Compara las clases bifurcadas con las versiones de PowerShell, teniendo en cuenta los destinos de compilación de .NET Core y .NET Standard. Se considerará terminado cuando se hayan abordado la deuda técnica, los errores de análisis conocidos y las restricciones de dependencias o del framework, sin romper el analizador.

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

Evaluación

Stack tecnológico
csharp, powershell
Área
tooling
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
20/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.