已开启
docs:更新modal组件br_rnoh0.72分支的中英文文档至v0.4.2模板 #28
docs:更新modal组件br_rnoh0.72分支的中英文文档至v0.4.2模板 #28
已开启
fangzhen创建于 8月18日
fangzhen
fangzhen成员
8月18日

docs:更新modal组件br_rnoh0.72分支的中英文文档至v0.4.2模板

likedislike
合并受阻
fangzhenfangzhen成员
8月18日 修改了pull request 的描述
openharmony_ci
openharmony_ci成员
8月18日 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
openharmony_ciopenharmony_ci成员
8月18日 添加了label:dco检查失败
cpf-manager
cpf-manager8月18日进行代码检视1
README.md
@@ -225,0 +227,4 @@
227+│ └─ modal.style.ts # 动画 / 样式相关
228+│ └─ types.ts # 类型定义(ModalProps、动画、方向等)
229+│ └─ utils.ts # 工具函数
230+│ └─ global.d.ts # 全局类型声明
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【一般】【基础代码问题】【可读性问题】目录结构文档了不存在的文件 global.d.ts

● 问题: 目录结构章节中列出了 src/global.d.ts(注释为"全局类型声明"),但该文件在仓库中并不存在。通过 GitCode API 验证,src/ 目录实际只包含 5 个文件:index.ts、modal.tsx、modal.style.ts、types.ts、utils.ts;直接请求 src/global.d.ts 返回 404。tsconfig.json 的 include 为 src/**/*,也未见对该文件的引用。

● 影响: 开发者按文档导航源码时找不到该文件,造成困惑;文档与实际代码不一致,降低文档可信度,后续维护者可能误以为该文件缺失是 bug 而去补建无意义的空文件。

● 建议: 从目录结构中删除 global.d.ts 行;若该文件本应存在,请补充提交该文件并在 tsconfig 中确认其作用。

likedislike
cpf-manager
cpf-manager8月18日进行代码检视1
README_en.md
@@ -225,0 +227,4 @@
227+│ └─ modal.style.ts # animation / styles
228+│ └─ types.ts # type definitions (ModalProps, animations, directions, etc.)
229+│ └─ utils.ts # utility functions
230+│ └─ global.d.ts # global type declarations
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【一般】【基础代码问题】【可读性问题】目录结构文档了不存在的文件 global.d.ts

● 问题: 目录结构章节中列出了 src/global.d.ts(注释为"global type declarations"),但该文件在仓库中并不存在。通过 GitCode API 验证,src/ 目录实际只包含 5 个文件:index.ts、modal.tsx、modal.style.ts、types.ts、utils.ts;直接请求 src/global.d.ts 返回 404。tsconfig.json 的 include 为 src/**/*,也未见对该文件的引用。

● 影响: 开发者按文档导航源码时找不到该文件,造成困惑;文档与实际代码不一致,降低文档可信度,后续维护者可能误以为该文件缺失是 bug 而去补建无意义的空文件。

● 建议: 从目录结构中删除 global.d.ts 行;若该文件本应存在,请补充提交该文件并在 tsconfig 中确认其作用。

likedislike
cpf-manager
cpf-manager8月18日进行代码检视2
README.md
已过期
@@ -43,0 +38,4 @@
38+| | 是否支持autolink | RN框架版本 |
39+| ------- | ---------------- | -------------- |
40+| ~13.0.2 | No | 0.72 |
41+| ~13.0.1@deprecated | No | 0.72 |
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【建议】【基础代码问题】【可读性问题】deprecated 版本号在版本表与 Link 表中不一致

● 问题: 同一份 README 中存在两处版本声明不一致:

  • 版本信息表(第 14 行)将 deprecated 版本写为 ~ 13.0.1-0.0.1@deprecated,npm 链接指向 https://www.npmjs.com/package/@react-native-oh-tpl/react-native-modal;
  • Link 表(第 41 行)将同一包写为 ~13.0.1@deprecated
    在 semver 下,13.0.1-0.0.1 是 13.0.1 的预发布版本,13.0.1 是正式版本,二者不是同一个版本号,~13.0.1-0.0.1~13.0.1 匹配的发布范围也不同。

● 影响: 用户无法从文档确定 @react-native-oh-tpl/react-native-modal 的实际安装版本号,两处自相矛盾降低文档可信度;按 Link 表执行 npm install react-native-modal@~13.0.1 与按版本表执行 ~13.0.1-0.0.1 安装得到的产物可能不一致。

● 建议: 核对 npm 上 @react-native-oh-tpl/react-native-modal 的实际发布版本,在版本信息表(第 14 行)与 Link 表(第 41 行)使用完全一致的版本号字符串。

likedislike
System
系统消息系统
8月18日 评论:

changed this line on b4f22d35 view diff detail

cpf-manager
cpf-manager8月18日进行代码检视1
README_en.md
@@ -43,0 +38,4 @@
38+| | Supported Autolink | Supported RN Version |
39+| ------- | ------------------ | -------------------- |
40+| ~13.0.2 | No | 0.72 |
41+| ~13.0.1@deprecated | No | 0.72 |
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【建议】【基础代码问题】【可读性问题】deprecated 版本号在版本表与 Link 表中不一致

● 问题: 同一份 README 中存在两处版本声明不一致:

  • 版本信息表(第 14 行)将 deprecated 版本写为 ~ 13.0.1-0.0.1@deprecated,npm 链接指向 https://www.npmjs.com/package/@react-native-oh-tpl/react-native-modal;
  • Link 表(第 41 行)将同一包写为 ~13.0.1@deprecated
    在 semver 下,13.0.1-0.0.1 是 13.0.1 的预发布版本,13.0.1 是正式版本,二者不是同一个版本号,~13.0.1-0.0.1~13.0.1 匹配的发布范围也不同。

● 影响: 用户无法从文档确定 @react-native-oh-tpl/react-native-modal 的实际安装版本号,两处自相矛盾降低文档可信度;按 Link 表执行 npm install react-native-modal@~13.0.1 与按版本表执行 ~13.0.1-0.0.1 安装得到的产物可能不一致。

● 建议: 核对 npm 上 @react-native-oh-tpl/react-native-modal 的实际发布版本,在版本信息表(第 14 行)与 Link 表(第 41 行)使用完全一致的版本号字符串。

likedislike
yangweiping
yangweiping8月18日进行代码检视2
README.md
已过期
@@ -43,0 +38,4 @@
38+| | 是否支持autolink | RN框架版本 |
39+| ------- | ---------------- | -------------- |
40+| ~13.0.2 | No | 0.72 |
41+| ~13.0.1@deprecated | No | 0.72 |
yangweiping
yangweiping8月18日评论:

改为13.0.1-0.0.1@deprecated, 保持一致

likedislike
System
系统消息系统
8月18日 评论:

changed this line on b4f22d35 view diff detail

fangzhenfangzhen成员
8月18日 强制推送  1 个提交:b4f22d35-docs:更新modal组件br_rnoh0.72分支的中英文文档至v0.4.2模板
openharmony_ci
openharmony_ci成员
8月18日 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
cpf-manager
cpf-manager8月18日进行代码检视1
README.md
@@ -156,3 +144,3 @@
156144> [!TIP] "Platform"列表示该属性在原三方库上支持的平台。
157145 
158-> [!TIP] "HarmonyOS Support"列为 yes 表示 HarmonyOS 平台支持该属性;no 则表示不支持;partially 表示部分支持。使用方法跨平台一致,效果对标 iOS 或 Android 的效果。
146+> [!TIP] "OpenHarmony Support"列为 yes 表示 OpenHarmony平台支持 该属性;no 则表示不支持;partially 表示部分支持。使用方法跨平台一致,效果对标 iOS 或 Android 的效果。
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【一般】【基础代码问题】【可读性问题】TIP提示引用的列名与属性表实际列头不一致

● 问题:README.md 第 146 行 TIP 提示引用了列名"OpenHarmony Support"(原文:"OpenHarmony Support"列为 yes 表示 OpenHarmony平台支持 该属性),但第 156 行属性表的实际列头是"HarmonyOS平台支持",表中不存在名为"OpenHarmony Support"的列。同时 TIP 中使用"OpenHarmony"前缀而列头使用"HarmonyOS"前缀,命名不一致。此问题由本 PR 重命名列头时引入:旧版本列头为"HarmonyOS Support"(与旧版 TIP 引用一致),新版本将列头改为中文"HarmonyOS平台支持"但未同步更新 TIP 中的列名引用。

● 影响:一般。用户按 TIP 提示查找名为"OpenHarmony Support"的列时找不到对应列,造成理解困惑;"HarmonyOS"与"OpenHarmony"前缀混用也降低文档一致性。

● 建议:将 TIP 中的列名引用更新为实际列头"HarmonyOS平台支持",例如改为:"HarmonyOS平台支持"列为 yes 表示 OpenHarmony 平台支持该属性;或将列头统一改为"OpenHarmony平台支持"并同步更新 TIP 中的引用。建议全文档统一使用"OpenHarmony"或"HarmonyOS"前缀之一。

likedislike
cpf-manager
cpf-manager8月18日进行代码检视1
README_en.md
@@ -156,3 +144,3 @@
156144> [!TIP] The **Platform** column indicates the platform where the properties are supported in the original third-party library.
157145 
158-> [!TIP] If the value of **HarmonyOS Support** is **yes**, it means that the HarmonyOS platform supports this property; **no** means the opposite; **partially** means some capabilities of this property are supported. The usage method is the same on different platforms and the effect is the same as that of iOS or Android.
146+> [!TIP] If the value of **HarmonyOS Support** is **yes**, it means that the OpenHarmony platform supports this property; **no** means the opposite; **partially** means some capabilities of this property are supported. The usage method is the same on different platforms and the effect is the same as that of iOS or Android.
cpf-manager
cpf-manager8月18日评论:

【AI-Review】【一般】【基础代码问题】【可读性问题】TIP提示引用的列名与属性表实际列头不一致

● 问题:README_en.md 第 146 行 TIP 提示引用了列名"HarmonyOS Support"(原文:If the value of HarmonyOS Support is yes),但第 156 行属性表的实际列头是"OpenHarmony Platform Support",表中不存在名为"HarmonyOS Support"的列。同时 TIP 中使用"HarmonyOS"前缀而列头使用"OpenHarmony"前缀,命名不一致。此问题由本 PR 重命名列头时引入:旧版本列头为"HarmonyOS Support"(与旧版 TIP 引用一致),新版本将列头改为"OpenHarmony Platform Support"但未同步更新 TIP 中的列名引用,仅将描述中的"HarmonyOS platform"改为了"OpenHarmony platform"。

● 影响:一般。用户按 TIP 提示查找名为"HarmonyOS Support"的列时找不到对应列,造成理解困惑;"HarmonyOS"与"OpenHarmony"前缀混用也降低文档一致性。

● 建议:将 TIP 中的列名引用更新为实际列头"OpenHarmony Platform Support",例如改为:If the value of OpenHarmony Platform Support is yes;并统一使用"OpenHarmony"或"HarmonyOS"前缀之一。

likedislike