Skip to content

Conversation

@ACaiCat
Copy link
Member

@ACaiCat ACaiCat commented Jul 11, 2025

  • 将PowellShell脚本中硬编码路径改为遍历所有版本的OTA目录

好的,这是翻译成中文的 pull request 摘要:

Sourcery 总结

启用自动生成所有已安装 OTAPI 版本的 OTAPI.xml 文档,通过扫描 NuGet 包目录而不是使用硬编码路径来实现。

新特性:

  • 递归扫描 .nuget/packages/otapi.upcoming 目录以查找所有 OTAPI.dll 文件。
  • 在 GitHub 工作流程中,在每个发现的 OTAPI.dll 文件旁边生成一个 OTAPI.xml 文件。
Original summary in English

Summary by Sourcery

Enable automated generation of OTAPI.xml documentation for all installed OTAPI versions by scanning the NuGet package directory instead of using a hard-coded path.

New Features:

  • Recursively scan the .nuget/packages/otapi.upcoming directory for all OTAPI.dll files.
  • Generate an OTAPI.xml file alongside each discovered OTAPI.dll during the GitHub workflow.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

@ACaiCat - 我已经查看了你的更改,它们看起来很棒!

AI 代理的提示
请解决此代码审查中的评论:
## 单独评论

### 评论 1
<location> `.github/workflows/dotnet.yml:45` </location>
<code_context>
+                Write-Host "已创建 OTAPI.xml 在: \$xmlPath"
+            }
         }
         Pause
         EOL
</code_context>

<issue_to_address>
暂停可能会在非交互式 CI 环境中引起问题。

删除 'Pause' 命令以防止 CI 作业因缺少用户输入而挂起。
</issue_to_address>

Sourcery 对于开源是免费的 - 如果你喜欢我们的评论,请考虑分享它们 ✨
帮助我更有用!请点击每个评论上的 👍 或 👎,我将使用反馈来改进你的评论。
Original comment in English

Hey @ACaiCat - I've reviewed your changes and they look great!

Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments

### Comment 1
<location> `.github/workflows/dotnet.yml:45` </location>
<code_context>
+                Write-Host "已创建 OTAPI.xml 在: \$xmlPath"
+            }
         }
         Pause
         EOL
</code_context>

<issue_to_address>
Pause may cause issues in non-interactive CI environments.

Remove the 'Pause' command to prevent CI jobs from hanging due to lack of user input.
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@ACaiCat ACaiCat merged commit 4d79212 into master Jul 11, 2025
1 check passed
@ACaiCat ACaiCat deleted the feature/install-xml-to-all-OTAPI branch July 11, 2025 03:57
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.

2 participants