Placeholders are substituted twice for stored commands
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 79/100
Piste de recherche
Lis set_task_command, run_task, le sélecteur d’historique et la vérification du nom de fichier ajoutés dans #44. Repère où la substitution des espaces réservés s’applique aux commandes enregistrées et aux commandes ponctuelles, puis vérifie le chemin de commande du sélecteur d’historique. Le travail est terminé lorsque les commandes enregistrées ne sont substituées qu’une seule fois, que les commandes ponctuelles fonctionnent toujours et que les noms de fichier contenant $E_FN sont acceptés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
A command that uses $E_FN is filled in when it is set (set_task_command), and then searched for $E_FN again when it runs (run_task). The history picker also passes an already filled-in command back through set_task_command.
For normal filenames the second pass finds nothing. If the filename itself contains the text $E_FN, the second pass replaces it again and produces a broken command. For example, a file named test_$E_FN.js would run echo 'test_'test_$E_FN.js'.js' instead of echo 'test_$E_FN.js'.
Since #44, such filenames are rejected with an error, so this cannot produce a broken or unsafe command. The proper fix is to fill in placeholders only once:
run_tasksubstitutes placeholders only for one-off commands; a stored command is already filled in.- The history picker stores the chosen entry directly instead of passing it through
set_task_commandagain.
Once that is done, the filename check added in #44 can be removed.
- Langage dominant
- Lua
- Étoiles
- 161
- Forks
- 12
- Merge moyen
- 25 min
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un 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 google/executor.nvim
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
google/executor.nvim#45 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
google/executor.nvim#38 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
google/executor.nvim#35 ·
-
[Feature Request] Option to run ExecutorShowDetail when the current task finishesPeut-être pris @jackfranklin l’a pris il y a 513 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
google/executor.nvim#33 · 9 commentaires ·
-
IMPORTANT: breaking changesOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
google/executor.nvim#26 · 3 commentaires ·
Toutes les issues de google/executor.nvim
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
teradata-labs/loom#624 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
lsp
Difficulté 1/5 1-3 heures Accessibilité débutants 76/100
neovim/neovim#42774 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour