Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Jun 29, 2023

让ci不会在fork的仓库上运行

@paddle-bot
Copy link

paddle-bot bot commented Jun 29, 2023

Thanks for your contribution!

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels Jun 29, 2023
@gouzil gouzil marked this pull request as draft June 29, 2023 08:20
@SigureMo
Copy link
Member

SigureMo commented Jun 29, 2023

actions/checkout 默认就有这功能吧?

比如 #220 https://github.com/PaddlePaddle/PaddleSOT/actions/runs/5409796119/jobs/9830376385?pr=220

image
  • 其中 9fff894 是 PR 的 hash
  • 其中 94fd654 是 develop 的 hash
  • 其中 30473f1 是 merge PR into develop 的 hash

@gouzil gouzil changed the title [ci] add merge check [ci] only run in PaddlePaddle/PaddleSOT repo Jun 29, 2023
@gouzil
Copy link
Member Author

gouzil commented Jun 29, 2023

actions/checkout 默认就有这功能吧?

比如 #220 https://github.com/PaddlePaddle/PaddleSOT/actions/runs/5409796119/jobs/9830376385?pr=220

image * 其中 [9fff894](https://github.com/PaddlePaddle/PaddleSOT/commit/9fff894f4d43b1f2f04dbe35d097cd897f2ca80b) 是 PR 的 hash * 其中 [94fd654](https://github.com/PaddlePaddle/PaddleSOT/commit/94fd654d92b51e0a84500caf15fa4af9b5410a8f) 是 develop 的 hash * 其中 [30473f1](https://github.com/PaddlePaddle/PaddleSOT/commit/30473f105f37bc218e874bcedc8417b9f3248c5e) 是 merge PR into develop 的 hash

是的是的, 我换了一个修

@gouzil gouzil marked this pull request as ready for review June 29, 2023 12:26
@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

这个看起来倒是没什么问题,只是这个是有什么需求吗?一般只有给 Fork 的 repo 提 PR 才会触发 CI?一旦我们以后需要在 fork 的 repo 里触发 CI 呢?

@gouzil
Copy link
Member Author

gouzil commented Jul 6, 2023

这个看起来倒是没什么问题,只是这个是有什么需求吗?一般只有给 Fork 的 repo 提 PR 才会触发 CI?一旦我们以后需要在 fork 的 repo 里触发 CI 呢?

可以看一下其他人的仓库, 比如 https://github.com/zrr1999/paddle-symbolic-trace, 可以发现merge后会再运行一次ci (不过我之前以为是会占用限时,现在看来好像不会)

@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

咦?我刚看你的 repo 是没有的啊

@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

啊,还真有

@gouzil
Copy link
Member Author

gouzil commented Jul 6, 2023

咦?我刚看你的 repo 是没有的啊

因为我手动把actions关了

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants