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

未关闭
#3,205 1 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
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 小时
30 天内合并 PR
29

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

github/vscode-codeql 的其他 Issue

查看 github/vscode-codeql 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。