Suggestion to inform users that default_root behaviour has changed.
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- Python
- Sterne
- 833
- Forks
- 96
- Ø Merge
- 1 Min.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 SublimeText/AdvancedNewFile
-
enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
SublimeText/AdvancedNewFile#196 · 1 Reaktion ·
-
enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
SublimeText/AdvancedNewFile#195 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
SublimeText/AdvancedNewFile#193 ·
-
enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
SublimeText/AdvancedNewFile#183 · 2 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
SublimeText/AdvancedNewFile#180 ·
Alle Issues in SublimeText/AdvancedNewFile
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
noontide-co/mainbranch#1211 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
[1.x Bug] run-operation --sql reports success but rolls back writes on transaction-capable adaptersOffentriage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 64/100
dbt-labs/dbt#16784 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug settings ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
DewaldOosthuizen/py-tray-command-launcher#162 ·
Maintainer antworten meist innerhalb von 1 Tag
-
run_all sleeps after the final job despite documenting delays between jobsEvtl. vergeben @Gambit-Checkmate hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100