[ENHANCEMENT] We need CI/CD test for PRs

[longlin10086]

As the title, now we need to accomplish auto CI/CD test for every PR.
Otherwise it would get some dirty commits to run CI/CD in production environment to know whether it can get appropriate performance.

[kowyo]

I used to add:

on:
    pull_request_target

into action file to trigger pr when opening a PR

[kowyo]

不太推荐使用单独的 debug 文件

可以在已有 workflow 文件加入 on: pull_request_target，参考 https://alampy.com/2024/04/20/gha-workflow-for-pull-request/#pull-request-target-事件 确保检出 pr 中的代码

[longlin10086]

不太推荐使用单独的 debug 文件

可以参考 https://alampy.com/2024/04/20/gha-workflow-for-pull-request/#pull-request-target-事件 确保检出 pr 中的代码

目前使用的是 pull-request 而非 pull-request-target，两者区别可以参考 GPT 回答：

pull_request_target 和 pull_request 事件在有 PR 时不会同时触发，它们是 互相独立的，并且根据你定义的工作流事件触发条件来决定是否触发。

具体来说：

pull_request 事件：当一个 Pull Request 被创建、同步（例如更新分支）或重新打开时触发。它运行在 源仓库 的上下文中。
pull_request_target 事件：与 pull_request 类似，当一个 Pull Request 被创建、同步或重新打开时触发，但它运行在 目标仓库 的上下文中，通常用于需要访问目标仓库的敏感信息（例如 secrets）或需要在目标仓库环境中执行的操作。

1. 源仓库（Source Repository）
源仓库是提交 PR 的 仓库，即发起 PR 的那个仓库。它通常是一个开发者或贡献者用来提交更改的仓库。
如果 PR 是从同一个仓库的不同分支中创建的，那么这个分支所在的仓库就是源仓库。
如果 PR 是从 forked 仓库（一个外部仓库）发起的，那么这个 forked 仓库就是源仓库。
源仓库就是你在 PR 中更改代码的地方，通常是提交者的个人仓库或 fork 的版本。

2. 目标仓库（Target Repository）
目标仓库是你希望将更改 合并 到的仓库，通常是原始仓库或其他被维护的仓库。
在 GitHub 上的 PR 流程中，目标仓库通常是项目的主要仓库或团队的仓库。
例如，如果你在 fork 仓库中进行了更改并发起了 PR，那么目标仓库就是原始仓库。
目标仓库通常是被更改的接收方，它包含你希望将更改合并进去的主分支（如 main 或 master）

使用 pull-request 可以保证 workflow 运行在发起 PR 的一方仓库内，这样测试的结果是直接反映 PR 代码里的运行结果，而非我们仓库内代码的运行结果。同时在另一配置文件内通过使用 ${GITHUB_TOKEN} 替代 ${scrects} 保证仓库安全性：

如果拉取请求存在合并冲突，工作流将不会在 pull_request 活动上运行。 必须先解决合并冲突。 相反，具有 pull_request_target 事件的工作流将运行，即使拉取请求存在合并冲突也是如此。 在使用 pull_request_target 触发器之前，应注意安全风险。 有关详细信息，请参阅 pull_request_target。

对于由 pull_request_target 事件触发的工作流，将授予 GITHUB_TOKEN 读/写存储库权限，除非指定了 permissions 密钥并且工作流可以访问机密，即使从分支触发也是如此。 虽然工作流程在拉取请求的基础上下文中运行，但你应该确保不在此事件中检出、生成或运行来自拉取请求的不受信任代码。 此外，任何缓存都与基本分支共享相同的作用域。 为帮助防止缓存中毒，如果缓存内容可能已更改，则不应保存缓存。

经测试，pull_request 也会为 GITHUB_TOKEN 授予读/写存储库权限。

[kowyo]

经测试，pull_request 也会为 GITHUB_TOKEN 授予读/写存储库权限。

pull_request 无法提供 PAT 的读取权限，在 dependent bot 开 pr 的时候 dependent bot 应该无法获取 PAT。详见 dependabot/dependabot-core#3253 。

如果个人用户的情况有读写权限那确实没问题。

不太推荐使用单独的 debug 文件

后续如果要修改 workflow 逻辑，应该直接在 course.yaml 需改，因此建议在原先的 course.yaml 加入 on: pull-request，在 commit and push 那一步使用 if 判断触发事件选择跳过。

[kowyo]

现在 dependent bot 开 pr 的时候无法获取 PAT，https://github.com/HITSZ-OpenAuto/hoa-moe/actions/workflows/course.yaml?query=is%3Afailure，可以考虑对此类情况加入 pull-request-target