PaddlePaddle / Paddle

PArallel Distributed Deep LEarning: Machine Learning Framework from Industrial Practice (『飞桨』核心框架,深度学习&机器学习高性能单机、分布式训练和跨平台部署)
http://www.paddlepaddle.org/
Apache License 2.0
22.29k stars 5.61k forks source link

[CodeStyle][Typos][C-[10-15]] Fix typo(CACH(4), catched(2), cuase(2), channles(1), channnel(2), cahr(1))。 #69552

Closed little1d closed 6 days ago

little1d commented 6 days ago

PR Category

User Experience

PR Types

Devs

Description

fit:

paddle-bot[bot] commented 6 days ago

你的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.

CLAassistant commented 6 days ago

CLA assistant check
All committers have signed the CLA.

CLAassistant commented 6 days ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

MrXnneHang commented 6 days ago

follwing可以删掉。

little1d commented 6 days ago

那我现在要重新修改提一个 PR 吗(新手保护 @MrXnneHang

MrXnneHang commented 6 days ago

@little1d 直接commit push到你 PR 的分支即可。另外你似乎得去那个机器人那边签署一下 CLA

little1d commented 6 days ago

当时一直commit 不上去,我参考了 #69483 的Git常见问题部分,那里推荐先checkout到本次修改的任务标识分支中。

image

后来问题是因为git commit是需要添加参数-n

little1d commented 6 days ago

@little1d 直接commit push到你 PR 的分支即可。另外你似乎得去那个机器人那边签署一下 CLA

好的,谢谢老师

MrXnneHang commented 6 days ago

当时一直commit 不上去,我参考了 #69483 的Git常见问题部分,那里推荐先checkout到本次修改的任务标识分支中。 image 后来问题是因为git commit是需要添加参数-n

我还从来没有用过-n,但似乎它会跳过一些git hooks?下次最好不要这么做,先把你commit -m提交不上去报的错误丢到FAQ里给我看看。 我也是同学(>_<)

little1d commented 6 days ago

好的同学,是因为报错里的内容没有我修改的内容,在社区大群里之前有同学遇到过这个问题,导师的回答是加一个-n参数,确保自己修改的文件通过 pre-commit 就可以了。但确实,是我的遗漏,我没仔细检查 pr,导致 pre-commit没过,不应该加这个参数。抱歉

little1d commented 6 days ago

@MrXnneHang 同学,可以再问您一个问题吗,我签署了 CLA,但是第二个机器人为啥还在pending

image
MrXnneHang commented 6 days ago

image

little1d commented 6 days ago

image

recheck 好多次了

luotao1 commented 6 days ago

第二个机器人为啥还在pending

因为第一次提交PR,需要人工后台approve,才能跑CI流程。请耐心等待下,稍后可以看到其他CI会跑了。

little1d commented 6 days ago

第二个机器人为啥还在pending

因为第一次提交PR,需要人工后台approve,才能跑CI流程。请耐心等待下,稍后可以看到其他CI会跑了。

好的,谢谢骆姐😘😘

little1d commented 6 days ago

CI,PR,Git相关的下次发到FAQ里。这里留着review就可以了。 image 这里显示你已经签署,可以尝试推送代码来触发CI,然后再把代码恢复回去再推送一次。

嗯嗯嗯,好的,第一次 pr,学到了。 ps: 您的解决办法也试过了,不起作用,应该是骆姐描述的问题