Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bugs in CI #97

Closed
wants to merge 18 commits into from
Closed

Fix bugs in CI #97

wants to merge 18 commits into from

Conversation

a793181018
Copy link
Contributor

No description provided.

1. Add code instances through the codelens feature to add references to inference services
2. Add custom code snippets through the codelens feature to add contextual references to inference services
2.add CodeSample model
3.add  FrameworkCodeFragment Model
4.add CodeSample and FrameworkCodeFragment manage Panel
5.create CodeSample and FrameworkCodeFragment support base
@zhengxs2018 zhengxs2018 requested a review from CGQAQ November 13, 2024 07:53
Copy link
Contributor

@CGQAQ CGQAQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the title is 'Fix bugs in CI', but what you've changed has nothing to do with CI

@CGQAQ CGQAQ requested a review from phodal November 13, 2024 07:57
@zhengxs2018 zhengxs2018 requested a review from CGQAQ November 18, 2024 00:47
@CGQAQ CGQAQ requested review from zhengxs2018 and removed request for CGQAQ November 18, 2024 00:49
2.add defult selected group
@CGQAQ CGQAQ removed their assignment Dec 2, 2024
@zhengxs2018
Copy link
Collaborator

非常感谢你的贡献!你的PR带来了一个很有创意的功能,我已经仔细审查了你提交的代码。

以下是一些需要优化的地方,希望你能进一步完善:

  1. 请注意,CodeSample 类与 Workspace 的命名存在冲突,请对其进行调整并释放命名空间。
  2. 新增的模板和命令需要考虑国际化兼容性,以便更好地服务于不同语言的用户。
  3. 发现 autodev.codelen.addCodeSample 命令有重复注册的情况,请移除多余的注册。
  4. 请确保变量命名的大小写一致性,并删除不必要的导入,同时对代码进行清理和格式化。

具体的修改建议我已经在文件评论中给出,欢迎你参考并提出讨论。

此外,我恢复了之前的提示语,并对图标进行了更新。你也可以访问 vscode 图标库 选择更适合的图标。

如果你有任何疑问或新的想法,欢迎随时联系 @CGQAQ 。再次对你的提交表示赞赏,期待你的进一步完善!👏

@phodal phodal closed this Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants