PowerShell / PowerShell/PSScriptAnalyzer
Update/standardize the internal custom `VariableAnalysis` classes
Nadie ha tomado este issue todavía.
- 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
VariableAnalysisclasses 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
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- 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