Open
fix: 启用 luacheck 全局变量等检查规则并清理告警 #33
masiyuAdd on 15 days ago
fix: 启用 luacheck 全局变量等检查规则并清理告警 #33
Open
Merged fromm0_64526648/clifix/luacheck-global-rulesintoopenUBMC/climain
Merge blocked
15 days ago Set WeliveWelove,DataSculptor,huanghan,aaronlee as approver
15 days ago Added label: sig/sig-interface
openUBMC-robot
15 days ago Comments:
15 days ago Comments:
欢迎来到 openUBMC 社区
Hey @m0_64526648 , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用,可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则,更多详情,请参阅机器人自助配置使用手册。
联系指引
有疑问可以联系 SIG: sig-interface ,
维护者是: @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是: @PLUS_ONE .


15 days ago Added label: ai-co-authored
102 messages are collapsed here。 See more
openubmc-ci-bot
10 days ago Comments:
10 days ago Comments:
CI已开始运行,查看流水线详情:流水线链接


10 days ago Added label: limit_1k_suc
10 days ago Removed label: ci-running
openubmc-ci-bot
10 days ago Comments:
10 days ago Comments:
| 任务名称 | 结果 |
|---|---|
| pre-commit检查 | ✅ SUCCESS |
| PR描述检查 | ✅ SUCCESS |
| 防投毒检查 | ✅ SUCCESS |
| 开源合规检查 | ✅ SUCCESS |
| 编译构建 | ✅ SUCCESS |
| 版本校验 | ✅ SUCCESS |
| ipmi修改 | ✅ SUCCESS |
| 修改行数检查 | ✅ SUCCESS |


10 days ago Added label: ci-successful
当前PR是否有AI参与:
PR功能描述 / 为什么需要这个合入:
本 PR 从
fix/luacheck-global-rules分支合并到main,包含 1 个提交,修改 2 个文件。当前仓库
.luacheckrc整体忽略了 111/411/221 等规则,导致全局变量污染、同作用域重复local等问题无法被检出。本次补齐规则并清理存量告警,使 Luacheck 在 pre-commit 中真正生效。变更类型
修改方案:
.luacheckrc:启用 111/531/532/411/221/612 并移除对应 ignore;增加测试用例类/打桩全局的名字模式忽略,补充注释说明。.luacheckrcsrc/lualib/ipmc.lua验证方案:
pre-commit run --all-files全量通过,Luacheck 告警为 0。测试建议:
.luacheckrc的抑制范围。自验证报告:
该PR关联的issue
(格式为 fixes #<issue号>, 或者resolves #<issue号>):
希望检视人员了解:
mds/service.json),有 CHANGELOG 的仓库同步补充了变更记录。关联 Issue:https://gitcode.com/openUBMC/bingo/issues/273