intersystems / intersystems/git-source-control

Configure() Prompt Improvements

オープン
#718 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
good first issue
主要言語
ObjectScript
スター
22
フォーク
14
平均マージ
1日 9時間
マージ済み PR(30日)
5

説明

While configuring Embedded Git with do ##class(SourceControl.Git.API).Configure() for the first time with @isc-pbarton , we found a few areas where the prompts could be clarified and improved:

1. 'The system's default mode' prompt should be _before_ the user setting for basic mode, not after. Additionally, the term 'system' here should be replaced with 'Namespace' as this sets basic/expert mode at the _Namespace_ level.
2. 'Branch from which Sync will merge changes, in basic mode': This prompt should not be asked if the plugin is in expert mode.
3. 'Define a namespace-level web application allowing access to multiple git repos across separate namespaces': In reality the purpose of this is to create a web application passthrough allowing you to edit the files in the git repo directly in VSCode, rather than just editing files in the SMP. This should be clarified in the prompt.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

SourceControl.Git.API.Configure() から開始して、設定プロンプトのフローを追跡します。namespace レベルのモードプロンプトが basic モードの設定より前に表示されること、expert モードでは sync-branch プロンプトがスキップされること、web-application プロンプトが VSCode でのリポジトリ編集について説明していることを確認します。初回の設定フローにこの 3 つの変更がすべて反映されていることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
git, vscode
領域
developer-experience, tooling
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。