Suggestion to inform users that default_root behaviour has changed.
まだ誰も着手していません。
評価
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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
- 主要言語
- Python
- スター
- 833
- フォーク
- 96
- 平均マージ
- 1分
- マージ済み PR（30日）
- 5
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
SublimeText/AdvancedNewFile のほかの issue
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
SublimeText/AdvancedNewFile#196 · リアクション 1 件 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
SublimeText/AdvancedNewFile#195 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
SublimeText/AdvancedNewFile#193 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
SublimeText/AdvancedNewFile#183 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
SublimeText/AdvancedNewFile#180 ·
SublimeText/AdvancedNewFile の issue をすべて見る
似ている issue
-
New Internshipオープンnew_internship
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
-
[BUG] Reports tab: "Unban" button tooltip shows raw `{{ip}}` placeholder instead of the IP addressオープンbug javascript ui
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
bunkerity/bunkerweb#4001 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
PedestrianDynamics/pyFDS-Evac#476 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
google/differential-privacy#516 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
adobe-fonts/source-serif#153 ·