Suggestion to inform users that default_root behaviour has changed.
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
Piste de recherche
Start in command_base.py and inspect how default_root and the per-command settings new_file_default_root, rename_file_default_root, and copy_file_default_root are read. Confirm the intended fallback when a per-command key is absent, while preserving explicitly configured per-command roots. Done means existing default_root configurations continue to work and individual roots can still be customized.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hi Scott,
I just figured out that default_root: project_root no longer works, and that as of 1.5.1 you need to set up individual settings for default_root in for each of the new, move, copy commands:
new_file_default_root
rename_file_default_root
copy_file_default_root
Here in command_base.py, could I suggest that you check for existence of the above keys in settings, and if not, then use the original default_root settings? This would allow the current installations to continue to work as is but allow fine tuning of roots if needed.
My Python has pretty much gone by the wayside as I haven't touched it since my 2 week stint last year, but something like this (excuse the Ruby-like code). I can't remember if self.settings.get returns nil or blank string, but something along these lines.
def _get_default_root(self):
root_setting_value = self.get_default_root_setting()
root_setting = self.settings.get(root_setting_value) || self.settings.get(DEFAULT_ROOT_SETTING)
if root_setting == DEFAULT_ROOT_SETTING:
return self.settings.get(DEFAULT_ROOT_SETTING)
return root_setting
Keep up the great work! I use ANF every single day I code!
Brian
- Langage dominant
- Python
- Étoiles
- 833
- Forks
- 96
- Merge moyen
- 1 min
- PR mergées (30 j)
- 5
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 SublimeText/AdvancedNewFile
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
SublimeText/AdvancedNewFile#196 · 1 réaction ·
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
SublimeText/AdvancedNewFile#195 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
SublimeText/AdvancedNewFile#193 ·
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
SublimeText/AdvancedNewFile#183 · 2 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
SublimeText/AdvancedNewFile#180 ·
Toutes les issues de SublimeText/AdvancedNewFile
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
EchoTools/nevr-runtime#651 ·
Les mainteneurs répondent en général sous 1 jour
-
bug frontend rt sev-3
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
noontide-co/mainbranch#1211 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
[1.x Bug] run-operation --sql reports success but rolls back writes on transaction-capable adaptersOuvertetriage
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
dbt-labs/dbt#16784 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour