已合并
!2737 Use IndentedPrinter to write idl #53
pavelmironchik创建于 2025年10月27日
!2737 Use IndentedPrinter to write idl #53
已合并
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 pavelmironchik 的贡献)openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
感谢提交 Pull Requests!
Thanks for submitting a pull request.


2025年10月27日 添加了label:dco检查成功
core/src/idl/dump.ts
@@ -356,0 +295,4 @@
295+ const initializer = idl.initializer === undefined
296+ ? ''
297+ : ' = ' + (type === IDLStringType.name
298+ ? `"${String(idl.initializer).replaceAll('"', "'")}"`
we need to think about not having " symbol in enum's initializers


2025年10月27日 解决了最后一个问题
2025年10月27日 通过审查
guozejun
2025年10月27日 评论:
2025年10月27日 评论:
FORCE_VERIFY


2025年10月27日 通过测试
openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
添加 测试通过 成功!


openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
guozejun
2025年10月27日 评论:
2025年10月27日 评论:
FORCE_VERIFY


2025年10月27日 通过测试
openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
添加 测试通过 成功!


openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
2025年10月27日 重置了审查状态
2025年10月27日 通过审查
guozejun
2025年10月27日 评论:
2025年10月27日 评论:
FORCE_VERIFY


2025年10月27日 通过测试
openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
添加 测试通过 成功!


openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
2025年10月27日 取消了草稿状态
2025年10月27日 修改标题为 “!2737 Use IndentedPrinter to write idl”,原标题为“[WIP] !2737 Use IndentedPrinter to write idl”
2025年10月27日 修改标题为 “!2737 Use IndentedPrinter to write idl”,原标题为“[WIP] !2737 Use IndentedPrinter to write idl”
2025年10月27日 设置为草稿状态
2025年10月27日 取消了草稿状态
pavelmironchik
2025年10月27日 评论:
2025年10月27日 评论:
FORCE_VERIFY


2025年10月27日 通过测试
openharmony_ci
2025年10月27日 评论:
2025年10月27日 评论:
添加 测试通过 成功!


2025年10月27日 合入了pull request,合并节点 SHA:f9f984da682310d1a0beac08283adf5a7c9db98f
core/src/idl/dump.ts
@@ -204,2 +102,2 @@
204- console.log(`Ignore ${idl.name}`)
205- return []
102+export class IDLWriter {
103+ constructor(protected printer: IndentedPrinter) {}
printer is only used privately in this class, so it probably could be created inside constructor, doesn't make sense to pass it from outside


一、内容说明(相关的Issue)
二、建议测试周期和提测地址
建议测试完成时间:xxxx.xx.xx
投产上线时间:xxxx.xx.xx
提测地址:CI环境/压测环境
测试账号:
三、变更内容
3.1 关联PR列表
3.2 数据库和部署说明
bundle exec xxx RAILS_ENV = production;没有写 "无"3.4 其他技术优化内容(做了什么,变更了什么)
3.5 废弃通知(什么字段、方法弃用?)
3.6 后向不兼容变更(是否有无法向后兼容的变更?)
四、研发自测点(自测哪些?冒烟用例全部自测?)
自测测试结论:
五、测试关注点(需要提醒QA重点关注的、可能会忽略的地方)
检查点:
接口测试:
性能测试:
并发测试:
其他: