Skip to content

docs: fix Java Maven coordinates to com.dingtalk:dingtalk-channel-sdk - #2

Merged
haofeng0705 merged 2 commits into
DingTalk-Real-AI:mainfrom
typefield:fix-java-maven-coords
Sep 20, 2026
Merged

haofeng0705 merged 2 commits into
DingTalk-Real-AI:mainfrom
typefield:fix-java-maven-coords

Conversation

@typefield

Copy link
Copy Markdown
Contributor

The cross-language install table (GUIDE.md:39, GUIDE.zh-CN.md:40) wrote the Java SDK coordinates as io.github.DingTalk-Real-AI:dingtalk-channel-sdk — not a resolvable artifact (and not the published coordinate).

Align it with the Java SDK migration in DingTalk-Real-AI/dingtalk-channel-sdk-java#1: Maven groupId com.dingtalk + package com.dingtalk.channel.

  • io.github.DingTalk-Real-AI:dingtalk-channel-sdk → com.dingtalk:dingtalk-channel-sdk

Docs-only change; no code affected.

The cross-language install table wrote the Java coordinates as
io.github.DingTalk-Real-AI:dingtalk-channel-sdk, which is neither the
current published artifact nor a resolvable groupId (groupIds cannot be
an org slug). Align with the Java SDK migration to
com.dingtalk:dingtalk-channel-sdk (DingTalk-Real-AI/dingtalk-channel-sdk-java#1),
which updates the POM groupId and all source packages to
com.dingtalk.channel.
@typefield

Copy link
Copy Markdown
Contributor Author

跨语言安装表中的 Java 坐标 io.github.DingTalk-Real-AI:dingtalk-channel-sdk 不是可解析的 artifact(也不是已发布坐标),对齐 java 仓库迁移后的 com.dingtalk:dingtalk-channel-sdk(仅 GUIDE.md:39 与 GUIDE.zh-CN.md:40 两处,纯文档改动,与 #3 互不冲突、可任意顺序合并)。@haofeng0705 请复查。

@haofeng0705 haofeng0705 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

复查结论:暂缓合入。

GUIDE.md:39、GUIDE.zh-CN.md:40 已改为 com.dingtalk:dingtalk-channel-sdk,但 Java SDK 的迁移 PR #1 尚未合入,Maven Central 当前也无法解析该构件。此时合入会让文档给出不可安装的依赖。

建议先完成 Java SDK #1、发布并验证新坐标,再合入本 PR;或者在文档中明确标注该坐标尚未发布。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@haofeng0705 haofeng0705 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

复查结论:请求修改。

GUIDE.md:39 和 GUIDE.zh-CN.md:40 的 Java Maven 坐标仍写为 io.github.DingTalk-Real-AI:dingtalk-channel-sdk,但本 PR 标题和提交说明是要改为 com.dingtalk:dingtalk-channel-sdk。当前文本与 Java SDK 迁移目标不一致。

另外,Java SDK #1 尚未合入,Maven Central 也查不到 com.dingtalk:dingtalk-channel-sdk 构件。建议:

  1. 将坐标统一改为 com.dingtalk:dingtalk-channel-sdk 并标注“待 Java SDK #1 发布后可安装”;或
  2. 保持旧坐标并修正 PR 标题/描述。

安装缺失依赖后 pytest 本地 108/108 通过。

@haofeng0705 haofeng0705 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

复查通过:已统一为 com.dingtalk:dingtalk-channel-sdk,并明确标注 0.1.0 尚待发布,英文与中文指南一致。

@haofeng0705
haofeng0705 merged commit 578bc5b into DingTalk-Real-AI:main Sep 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants