-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
ci: push image to GHCR for pull_request event #4588
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4588 +/- ##
============================================
+ Coverage 61.04% 61.14% +0.09%
- Complexity 2566 2580 +14
============================================
Files 374 374
Lines 13168 13221 +53
Branches 938 944 +6
============================================
+ Hits 8039 8084 +45
- Misses 4676 4684 +8
Partials 453 453 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Reverts #4588 According to https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions#using-secrets-in-a-workflow, we cannot obtain secrets in pull requests, so that we are not able to push docker image for every pull request. ```release-note None ```
What type of PR is this?
What this PR does / why we need it:
对于 Pull Request 触发的 Action 构建得到的镜像也 Push 到 GHCR 仓库中,并使用 pr-{PR编号} 格式的标签。
依赖 halo-sigs/actions#18
Which issue(s) this PR fixes:
Close #4315
Special notes for your reviewer:
Does this PR introduce a user-facing change?