Unify command execution in sudo.ps1
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 55/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- powershell
- Domaine
- cli
Piste de recherche
Commencez dans sudo.ps1 et localisez les trois blocs d’exécution répétés de SUDOEXE. Comparez leurs chemins de débogage et normal pour PowerShell 7, puis centralisez ce comportement dans un helper et remplacez les trois points d’appel tout en préservant le comportement d’exécution existant. Le travail est terminé lorsque les futures modifications de l’exécution des commandes peuvent être effectuées à un seul endroit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description of the new feature / enhancement
There are 3 places in the sudo.ps1 script that execute SUDOEXE in the following manner:
if (($psversiontable.psversion.major -eq 7) -and ($__SUDO_DEBUG -eq $true)) {
Trace-Command -PSHOST -name param* -Expression { & $SUDOEXE ... }
}
else {
& $SUDOEXE ...
}
Maybe export this logic to a function Run-Sudo or something along those lines?
Scenario when this would be used?
Any future change, bugfix, or improvement to the sudo executable or sudo script that need to be addressed at the points of command execution will only need to be applied in 1 place.
Supporting information
No response
- Langage dominant
- Rust
- Étoiles
- 5.9k
- Forks
- 180
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- 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 microsoft/sudo
-
Invalid translation in helpOuverteIssue-Bug Needs-Triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 74/100
-
Issue-Bug Needs-Triage
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
-
Issue-Feature Needs-Triage
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
-
Issue-Bug Needs-Triage
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
-
Issue-Bug Needs-Triage
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Toutes les issues de microsoft/sudo
Issues similaires
-
request-device panics the host on Device(Lost), Device(OutOfMemory) and other unmapped errorsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
wasi-gfx/wasi-gfx-runtime#152 ·
-
`rtk ruff -- <path>` drops the leading `--`, so `rtk ruff -- check` lints the whole directoryOuvertearea:cli bug good first issue priority:medium
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
rtk-ai/rtk#4702 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
aaif-goose/goose#13031 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour