CodeQL model editor: Support adding additional flows to already modeled methods

オープン
#3,205 コメント 1 件 リアクション 1 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
45/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
typescript, vscode
領域
devtools

調査の方向性

ファイルもテストも指定されていません。まず、"Method already modeled" エントリをレンダリングする CodeQL モデルエディター UI を見つけ、要求された動作を確認してください。既存のフローのコントロールを無効にしたまま、追加のフロー用に "+" コントロールを表示します。

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

説明

enhancement VSCode

Is your feature request related to a problem? Please describe.
When a method is already modeled, the entry shows only "Method already modeled" and there is no way to add additional flows.
This means:

  • you can't see the details of the existing model (type, input, output, kind)
  • you can't add additional flows because there is no "+" button

Describe the solution you'd like
The "+" button should also be shown for already modeled methods.

Side notes:

  • It might also be nice if instead of "Method already modeled", it would show how the method is modeled (type, input, output, kind) to notice issues or missing flow. Possibly also with multiple lines if there are multiple flows.
    But the dropdowns and the delete button should be disabled for these existing flows.
  • When "Hide modeled methods" is unselected, it might be useful to show the modeled methods intermixed with the un-modeled methods.
    Currently they are all shown as one group at the end of that API section, so you have to jump back and forth to see if / how the methods of a class are modeled in case some are already modeled and some are not.
    (Some users might prefer the existing grouping though?)

Describe alternatives you've considered
none

主要言語
TypeScript
スター
539
フォーク
240
平均マージ
2日 6時間
マージ済み PR(30日)
29

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

github/vscode-codeql のほかの issue

github/vscode-codeql の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

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

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