已合并
!2644 clang edits #55
pavelmironchik创建于 2025年10月27日
!2644 clang edits #55
已合并
pavelmironchik创建于 2025年10月27日
pavelmironchik
pavelmironchik成员
2025年10月27日

一、内容说明(相关的Issue)

二、建议测试周期和提测地址

建议测试完成时间:xxxx.xx.xx
投产上线时间:xxxx.xx.xx
提测地址:CI环境/压测环境
测试账号:

三、变更内容

  • 3.1 关联PR列表

  • 3.2 数据库和部署说明

    1. 常规更新
    2. 重启unicorn
    3. 重启sidekiq
    4. 迁移任务:是否有迁移任务,没有写 "无"
    5. rake脚本:bundle exec xxx RAILS_ENV = production;没有写 "无"
  • 3.4 其他技术优化内容(做了什么,变更了什么)

    • 重构了 xxxx 代码
    • xxxx 算法优化
  • 3.5 废弃通知(什么字段、方法弃用?)

  • 3.6 后向不兼容变更(是否有无法向后兼容的变更?)

四、研发自测点(自测哪些?冒烟用例全部自测?)

自测测试结论:

五、测试关注点(需要提醒QA重点关注的、可能会忽略的地方)

检查点:

需求名称 是否影响xx公共模块 是否需要xx功能 需求升级是否依赖其他子产品
xxx 需要 不需要

接口测试:

性能测试:

并发测试:

其他:

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 pavelmironchik 的贡献)
openharmony_ci
openharmony_ci成员
2025年10月27日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
2025年10月27日 添加了label:dco检查成功
pavelmironchik
pavelmironchik成员2025年10月27日进行代码检视1
tools/clang-format/package.json
@@ -0,0 +5,4 @@
5+ "license": "Apache-2.0",
6+ "keywords": [],
7+ "config": {
8+ "sig_path": "../../external/interop/src/cpp/ ../../external/incremental/graphics/native/platform/cpp ../../external/subset/arkoala-arkts/framework/native/src/generated ../../external/libarkts/native/src",
pavelmironchik
pavelmironchik2025年10月27日评论:

need to think carefully about relative paths, migration to repo sync is planned to be initialized soon and we must not forget about all of this

likedislike
pavelmironchikpavelmironchik成员
2025年10月27日 解决了最后一个问题
kolegovilya
kolegovilya2025年10月27日进行代码检视1
tools/clang-format/apply-correct-format.sh
@@ -0,0 +1,0 @@
1+#!/bin/bash
kolegovilya
kolegovilya2025年10月27日评论:

Bash script is not really "cross-platform" solution. Maybe one day we could think about rewrite this script to something more windows-friendly.

likedislike
kolegovilya
kolegovilya2025年10月27日进行代码检视2
tools/clang-format/package.json
@@ -0,0 +5,4 @@
5+ "license": "Apache-2.0",
6+ "keywords": [],
7+ "config": {
8+ "sig_path": "../../external/interop/src/cpp/ ../../external/incremental/graphics/native/platform/cpp ../../external/subset/arkoala-arkts/framework/native/src/generated ../../external/libarkts/native/src",
kolegovilya
kolegovilya2025年10月27日评论:

sig_path name could be misleading. Now we have a lot of names to one thing: external/sig/koala_project etc. I thing we should align in naming and use one name convension in our projects.

likedislike
peterz_5686
peterz_5686
2025年10月27日 评论:
likedislike
guozejunguozejun成员
2025年10月28日 通过审查
pavelmironchikpavelmironchik成员
2025年10月29日 取消了草稿状态
pavelmironchikpavelmironchik成员
2025年10月29日 修改标题为 “!2644 clang edits”,原标题为“[WIP] !2644 clang edits”
pavelmironchik
pavelmironchik成员
2025年10月29日 评论:

FORCE_VERIFY

likedislike
openharmony_ciopenharmony_ci成员
2025年10月29日 通过测试
openharmony_ci
openharmony_ci成员
2025年10月29日 评论:

添加 测试通过 成功!

likedislike
openharmony_ciopenharmony_ci成员
2025年10月29日 合入了pull request,合并节点 SHA:be335bb10cf2d82b3ca5fc9666726627703868ef