Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CodeStyle][py36] 修改部分关于 py36 的内容 #5460

Merged
merged 6 commits into from
Dec 1, 2022

Conversation

gsq7474741
Copy link
Contributor

有些地方不确定要不要改,这个文档修改可能需要多讨论下

related works

@paddle-bot
Copy link

paddle-bot bot commented Nov 29, 2022

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-5460.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:[Beta]飞桨文档预览工具

@gsq7474741
Copy link
Contributor Author

@SigureMo

@SigureMo SigureMo self-requested a review November 29, 2022 20:15
@SigureMo SigureMo self-assigned this Nov 29, 2022
docs/install/Tables_en.md Show resolved Hide resolved
docs/install/compile/arm-compile.md Show resolved Hide resolved
docs/install/FAQ.md Show resolved Hide resolved
docs/install/FAQ_en.md Outdated Show resolved Hide resolved
docs/install/FAQ.md Outdated Show resolved Hide resolved
docs/install/compile/macos-compile.md Outdated Show resolved Hide resolved
docs/install/compile/linux-compile.md Outdated Show resolved Hide resolved
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

其余 LGTM~

docs/install/pip/macos-pip.md Outdated Show resolved Hide resolved
@SigureMo
Copy link
Member

SigureMo commented Nov 30, 2022

另外问一下这个 PR 是清理(3.5/3.6 的)全部还是只是一部分呀?

@gsq7474741
Copy link
Contributor Author

另外问一下这个 PR 是清理(3.5/3.6 的)全部还是只是一部分呀?

我全文搜索了和3.5 3.6相关的内容,修改了我认为可能需要修改的部分,其余部分可能需要讨论一下看看哪里需要修改

@SigureMo
Copy link
Member

我全文搜索了和3.5 3.6相关的内容,修改了我认为可能需要修改的部分,其余部分可能需要讨论一下看看哪里需要修改

好哒,如果有的话可以在之后 PR 讨论(或者讨论后提 PR)~

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTM~

麻烦 @luotao1 确认下修改是否有问题~

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

Successfully merging this pull request may close these issues.

3 participants