Refactor PowerShell adapter module code for maintainability
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 42/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- powershell
- Domaine
- tooling
Piste de recherche
Commencez par localiser le script d’entrée partagé ainsi que les modules psDscAdapter et win_psDscAdapter, puis mettez en correspondance leurs implémentations dupliquées et les tests d’acceptation existants. Définissez les fonctions partagées, leurs paramètres et leurs données de retour, ainsi que le périmètre des tests unitaires des fonctions privées avant la restructuration ; le travail est terminé lorsque les deux adaptateurs utilisent le module commun et que les nouveaux tests unitaires couvrent le comportement extrait.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary of the new feature / enhancement
As a maintainer of DSC, I want to be able to readily navigate, understand, and update the PowerShell adapter code so that I can effectively contribute and troubleshoot the adapters.
Currently, the PowerShell adapter implementations share an entrypoint script but define two separate modules with shared implementations for functionality duplicated in both modules.
Additionally, the code organization is a bit difficult to follow and uses long complex statements repeatedly through the code which could potentially be encapsulated into small functions for easier testing and debugging.
Proposed technical implementation details (optional)
I propose that we:
- Extract common functions into a
common_psDscAdaptermodule that bothpsDscAdapterandwin_psDscAdaptercan depend on. - Standardize and at least minimally document the functions - their parameters, usage, and return data - for easier cognitive loading for maintainers. Even though this code won't be seen or used directly by end users, this reference information can help maintainers and new contributors.
- Examine the implementation for code that can be extracted into useful functions for the purposes of testing and making the overall implementation easier to read and follow.
- Add unit testing for the private functions to verify behavior and make maintenance easier and safer. We currently rely almost exclusively on acceptance tests that invoke the adapter code through DSC, which is necessarily slower and involves more levels of indirection.
- Langage dominant
- Rust
- Étoiles
- 541
- Forks
- 78
- Merge moyen
- 2 j 22 h
- PR mergées (30 j)
- 17
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire 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.
Autres issues de PowerShell/DSC
-
Failure to parse a resource manifest is logged as INFO instead of WARNPeut-être pris @SteveL-MSFT l’a pris il y a 4 jours. OuverteIssue-Bug Need-Review
Difficulté 2/5 1-3 heures Accessibilité débutants 69/100
PowerShell/DSC#1765 · 1 personne assignée ·
Les mainteneurs répondent en général sous 2 jours
-
Tracing categoriesPeut-être pris @michaeltlombardi l’a pris il y a 6 jours. OuverteArea-OpenTelemetry Issue-Enhancement Needs Triage
PowerShell/DSC#1767 · 1 réaction · 1 personne assignée ·
Les mainteneurs répondent en général sous 2 jours
-
Issue-Enhancement Needs Triage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
PowerShell/DSC#1763 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 2 jours
-
Issue-Enhancement Needs Triage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
PowerShell/DSC#1760 ·
Les mainteneurs répondent en général sous 2 jours
-
Hardening with dsc 3 ?Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. OuverteIssue-Enhancement Needs Triage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 12/100
PowerShell/DSC#1755 · 7 commentaires ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de PowerShell/DSC
Issues similaires
-
accounts bug decoding
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
wboayue/rust-ibapi#1088 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
`mise doctor` Shows Not Installed Lazy Tools as ProblemsPeut-être pris @jdx l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Les mainteneurs répondent en général sous 1 jour
-
area:kernel needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
bherbruck/solvecraft#7179 · 1 commentaire ·
-
priority: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 61/100
storytold/photocraft#9704 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour