-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Revert "[ci]:Add a token to fix the 403 error" #10883
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
Conversation
This reverts commit e458c0d.
🤖 CI Test ResultsLast Updated: 2025-11-02T09:07:45.142Z Workflow Status:
✅ Success | ❌ Failure | 🟡 In Progress This comment is automatically updated as CI workflows complete. |
🤖 CI Test ResultsLast Updated: 2025-11-02T09:07:56.055Z Test Spec & Results:✅ Success | ❌ Failure | 🟠 Queued | 🟡 Progress | ⏭️ Skipped | Overall Summary
Detailed Results✅ utest_auto_run - SuccessWorkflow: utest_auto_run Jobs:
🤖 This comment is automatically generated and updated by the CI system. |
|
两点修改建议: 1.ci运行成功可以不用抛出日志 |
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
📌 Code Review Assignment🏷️ Tag: workflowReviewers: @Rbb666 @kurisaW @supperthomas Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-11-02 16:59 CST)
📝 Review Instructions
|
|
目前是fail的,revert之后也是fail的。 @kurisaW 有空看看如何修复一下token |
|
实在不行,把之前的AI写的PR 先一起 revert掉 |
|
先把这个CI的触发条件给关掉吧,可以先在下游仓库测试通过后,然后再重新提一笔 |
|
@supperthomas I've opened a new pull request, #10884, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
手动revert或者 @Rbb666 先把comment功能的ci 先disable掉。不要报错。 |
|
这个PR禁用了该CI运行,然后这个可以先不用revert,可以先关闭PR |
Reverts #10878