Open
fix: 启用 luacheck 全局变量等检查规则并清理告警 #33
fix: 启用 luacheck 全局变量等检查规则并清理告警 #33
Open
masiyuAdd on 15 days ago
masiyu
masiyuCollaborators
15 days ago

当前PR是否有AI参与:

    1. AI Agent 平台: opencode
    2. AI 模型: deepseek-v4.1-flash
    3. Prompt上下文: 按照 network_adapter 模式启用 luacheck 规则并清理仓库内告警

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;增加测试用例类/打桩全局的名字模式忽略,补充注释说明。
  • 测试代码统一忽略 111/411/221(全局打桩对象与同作用域重复 local)。
  • 源码重复 local、未赋值变量等告警按需修复;可能改变语义或无意义的场景采用定向抑制。
  • 涉及文件:
    • .luacheckrc
    • src/lualib/ipmc.lua

验证方案:

  • pre-commit run --all-files 全量通过,Luacheck 告警为 0。

测试建议:

  • 关注被修改源文件的逻辑等价性(仅调整局部变量声明)以及 .luacheckrc 的抑制范围。

自验证报告:

该PR关联的issue

(格式为 fixes #<issue号>, 或者resolves #<issue号>):

希望检视人员了解:

  • 测试用例类全局声明未改为 local(luaunit 依赖全局表收集用例),通过忽略规则放行。
  • 已按 patch 规则升级版本号(mds/service.json),有 CHANGELOG 的仓库同步补充了变更记录。

关联 Issue:https://gitcode.com/openUBMC/bingo/issues/273

likedislike
Merge blocked
masiyumasiyuCollaborators
15 days ago Created pull request, commit 9bc64b2d
openUBMC-robotopenUBMC-robotCollaborators
15 days ago Set WeliveWelove,DataSculptor,huanghan,aaronlee as approver
openUBMC-robotopenUBMC-robotCollaborators
15 days ago Added label: sig/sig-interface
openUBMC-robot
openUBMC-robotCollaborators
15 days ago Comments:

欢迎来到 openUBMC 社区

Hey @m0_64526648 , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 说明文档 或 论坛 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。您可以自助配置此仓库的 PR 合并规则,更多详情,请参阅机器人自助配置使用手册。

联系指引

有疑问可以联系 SIG: sig-interface ,
维护者是: @DataSculptor, @WeliveWelove, @aaronlee, @huanghan ,
优秀贡献者是: @PLUS_ONE .

likedislike
openUBMC-robotopenUBMC-robotCollaborators
15 days ago Added label: ai-co-authored
102 messages are collapsed here。 See more
openubmc-ci-botCollaborators
10 days ago Comments:

CI已开始运行,查看流水线详情:流水线链接

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