PowerShell / PowerShell/PSScriptAnalyzer
Update/standardize the internal custom `VariableAnalysis` classes
Personne n'a encore pris cette issue.
- Langage dominant
- C#
- Étoiles
- 2.2k
- Forks
- 414
- Merge moyen
- 13 h 1 min
- PR mergées (30 j)
- 2
Description
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)
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par lire la discussion liée dans PowerShell/PSScriptAnalyzer#1277 et identifiez les classes internes de VariableAnalysis ainsi que leurs dépendances. Comparez les classes forkées avec les versions de PowerShell en tenant compte des cibles de compilation .NET Core et .NET Standard. Le travail est considéré comme terminé lorsque la dette technique, les bogues d’analyse connus et les contraintes liées aux dépendances ou au framework sont traités sans casser l’analyseur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- csharp, powershell
- Domaine
- tooling
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 20/100