Skip to content

[WIP] Fix CI for duplicated checks in PR#15

Closed
Copilot wants to merge 1 commit intomasterfrom
copilot/fix-ci-duplicate-checks
Closed

[WIP] Fix CI for duplicated checks in PR#15
Copilot wants to merge 1 commit intomasterfrom
copilot/fix-ci-duplicate-checks

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026

  • Analyze the issue: Both CI workflows (ci.yml and build.yml) trigger on both push and pull_request events
  • Fix ci.yml to prevent duplicate runs on PRs
  • Fix build.yml to prevent duplicate runs on PRs
  • Verify workflow configurations are correct
  • Validate YAML syntax
  • Request code review
  • Run security checks
Original prompt

This section details on the original issue you should resolve

<issue_title>[bug] needs to fix CI for strange duplicated checks in PR</issue_title>
<issue_description>The following duplicated CI checks ran in this open PR: #13

[Build / Running Go tests (pull_request)](https://github.com/casbin/zap-logger/actions/runs/21559242983/job/62120636290?pr=13)
Build / Running Go tests (pull_request)Successful in 39s
[Build / Running Go tests (push)](https://github.com/casbin/zap-logger/actions/runs/21559242543/job/62120636229?pr=13)
Build / Running Go tests (push)Successful in 37s
[CI / semantic-release (pull_request)](https://github.com/casbin/zap-logger/actions/runs/21559242970/job/62120651038?pr=13)
CI / semantic-release (pull_request)Successful in 3s
[CI / semantic-release (push)](https://github.com/casbin/zap-logger/actions/runs/21559242538/job/62120649604?pr=13)
CI / semantic-release (push)Successful in 21s
[CI / test (pull_request)](https://github.com/casbin/zap-logger/actions/runs/21559242970/job/62120636195?pr=13)
CI / test (pull_request)Successful in 24s
[CI / test (push)](https://github.com/casbin/zap-logger/actions/runs/21559242538/job/62120636183?pr=13)
CI / test (push)Successful in 21s

I think for an open state PR, it should not run "XXX (push)" checks, right? It should run for the git push after PR merge?
Need to fix the CI.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@CLAassistant
Copy link

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.

@nomeguy nomeguy marked this pull request as ready for review February 1, 2026 08:06
@nomeguy nomeguy closed this Feb 1, 2026
Copilot AI requested a review from nomeguy February 1, 2026 08:08
Copilot stopped work on behalf of nomeguy due to an error February 1, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] needs to fix CI for strange duplicated checks in PR

3 participants