Skip to content

[CodeStyle][DocFormat][119] Use pycon marker in fleet utils docs#77983

Merged
SigureMo merged 3 commits intoPaddlePaddle:developfrom
ShigureNyako:docs/migrate-docstring-highlight-markers-119
Feb 19, 2026
Merged

[CodeStyle][DocFormat][119] Use pycon marker in fleet utils docs#77983
SigureMo merged 3 commits intoPaddlePaddle:developfrom
ShigureNyako:docs/migrate-docstring-highlight-markers-119

Conversation

@ShigureNyako
Copy link
Contributor

PR Category

User Experience

PR Types

Docs

Description

Migrate docstring example markers from .. code-block:: python to .. code-block:: pycon in fleet utils docs.

Updated files:

  • python/paddle/distributed/fleet/utils/hybrid_parallel_inference.py
  • python/paddle/distributed/fleet/utils/__init__.py

Related PRs:

是否引起精度变化

@ShigureNyako
Copy link
Contributor Author

自检完成:本 PR 仅将 docstring 中的 .. code-block:: python 迁移为 .. code-block:: pycon,未改动示例语义。

@SigureMo 辛苦帮忙 review 🙏

@paddle-bot
Copy link

paddle-bot bot commented Feb 19, 2026

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Feb 19, 2026
@SigureMo
Copy link
Member

codestyle 没过,不给 review

@SigureMo
Copy link
Member

另外怎么改的文件数量越来越少了啊,任务描述不是写了一个 PR 5-10 个文件么?之后 PR 可以按照这个数量来

@ShigureNyako
Copy link
Contributor Author

已处理 codestyle 失败并推送 commit 11941a19cbruff format)。

关于文件数量建议已收到:后续新 PR 会按任务描述控制在 5-10 个文件,保持主题聚焦且避免与已有 PR 重叠。

这边先触发失败检查重跑,辛苦 @SigureMo 再看一眼 🙏

/re-run all-failed

@ShigureNyako
Copy link
Contributor Author

已按本轮 review 5 条建议全部完成修复并推送 commit f5502238dc

  • __init__.py 中 4 处示例调用补齐 trailing comma
  • hybrid_parallel_inference.py 中 1 处示例调用补齐 trailing comma

5 个 review 线程已逐条回复并 resolve。辛苦 @SigureMo 再看一眼 🙏

/re-run all-failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers skip-ci: all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments