Refactor PowerShell adapter module code for maintainability
Maintainer antworten meist innerhalb von 2 Tagen
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 42/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- powershell
- Bereich
- tooling
Rechercherichtung
Beginne damit, das gemeinsame Einstiegsskript sowie die Module psDscAdapter und win_psDscAdapter zu finden, und bilde anschließend ihre doppelten Implementierungen und vorhandenen Akzeptanztests ab. Definiere vor der Umstrukturierung die gemeinsamen Funktionen, ihre Parameter und Rückgabedaten sowie den Umfang der Unit-Tests für private Funktionen; abgeschlossen ist die Arbeit, wenn beide Adapter das gemeinsame Modul verwenden und die neuen Unit-Tests das extrahierte Verhalten abdecken.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Rust
- Sterne
- 526
- Forks
- 76
- Ø Merge
- 4 T. 20 Std.
- Gemergte PRs (30 T.)
- 24
Entwicklungsumgebung
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus PowerShell/DSC
-
Issue-Enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 38/100
PowerShell/DSC#1729 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Feature Request: Support dsc functions in executable argsEvtl. vergeben @SteveL-MSFT hat das vor 4 Tagen übernommen. OffenIssue-Enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 62/100
PowerShell/DSC#1722 · 4 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Dev-UX Needs Triage
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
PowerShell/DSC#1694 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Issue-Enhancement Needs Triage
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 38/100
PowerShell/DSC#1683 · 5 Kommentare ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Issue-Enhancement Needs Triage
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
PowerShell/DSC#1673 ·
Maintainer antworten meist innerhalb von 2 Tagen
Ähnliche Issues
-
area: dogs area: lookout bug security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
Maintainer antworten meist innerhalb von 2 Tagen
-
documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
OpenDevicePartnership/ina4230#31 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:breg criticality:p3 documentation triage:needs-implementation
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 86/100
registrystack/registry-stack#1713 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
posit-dev/ggsql#565 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag