Skip to content

ci: fix Gitee mirror workflow triggers#5778

Queued
njzjz-bot wants to merge 2 commits into
deepmodeling:masterfrom
njzjz-bothub:fix/skip-gitee-mirror-in-merge-queue
Queued

ci: fix Gitee mirror workflow triggers#5778
njzjz-bot wants to merge 2 commits into
deepmodeling:masterfrom
njzjz-bothub:fix/skip-gitee-mirror-in-merge-queue

Conversation

@njzjz-bot

@njzjz-bot njzjz-bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • run the Docker-based Gitee mirror action on ubuntu-latest, where a Docker daemon is available
  • prevent the mirror workflow from running for GitHub merge-queue gh-readonly-queue/** refs
  • use push as the single trigger, preserving branch and tag mirroring while avoiding duplicate create/delete runs

Validation

  • git diff --check
  • workflow trigger invariants checked locally (all branches/tags included; merge queue excluded)

Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.6-terra)

Summary by CodeRabbit

  • Chores
    • Improved repository mirroring by expanding triggers to cover both branch and tag pushes while excluding refs that shouldn’t be mirrored.
    • Adjusted mirroring behavior for delete events to suppress merge-queue-related ephemeral ref deletions while keeping other deletions guarded.
    • Updated the mirroring runner to use ubuntu-latest for more reliable Docker-based execution.

Authored by OpenClaw (model: custom-chat-jinzhezeng-group/gpt-5.6-terra)
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 70447fe7-bbe3-4e51-ae3c-cbb257dd0c7e

📥 Commits

Reviewing files that changed from the base of the PR and between c838a0e and aea80e3.

📒 Files selected for processing (1)
  • .github/workflows/mirror_gitee.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/mirror_gitee.yml

📝 Walkthrough

Walkthrough

The Gitee mirroring workflow consolidates branch and tag push filters, excludes selected ephemeral and automation refs, refines delete-event handling, and changes the mirror job runner to ubuntu-latest for Docker support.

Changes

Gitee mirror workflow

Layer / File(s) Summary
Push trigger filtering
.github/workflows/mirror_gitee.yml
Push events now match branches and tags while excluding merge-queue and specified automation refs; the create stub is removed and delete handling is retained.
Docker-capable runner configuration
.github/workflows/mirror_gitee.yml
The mirror job switches to ubuntu-latest, suppresses merge-queue ephemeral deletions, and documents its Docker daemon requirement.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: wanghan-iapcm

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: fixing the Gitee mirror workflow triggers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@njzjz njzjz requested a review from wanghan-iapcm July 12, 2026 18:08
@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.48%. Comparing base (a152b3b) to head (aea80e3).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5778      +/-   ##
==========================================
- Coverage   79.70%   79.48%   -0.22%     
==========================================
  Files        1020     1023       +3     
  Lines      116360   117415    +1055     
  Branches     4305     4328      +23     
==========================================
+ Hits        92744    93331     +587     
- Misses      22076    22526     +450     
- Partials     1540     1558      +18     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Review found one blocking correctness issue in deletion handling; see the inline comment. Because this review is being submitted by the PR author account, GitHub only permits a comment review rather than a formal request-changes review.

Coding agent: Codex
Codex version: codex-cli 0.144.1
Model: gpt-5.6-sol
Reasoning effort: xhigh

Comment thread .github/workflows/mirror_gitee.yml Outdated
Restore the delete event so branch and tag removals are mirrored immediately. Filter merge-queue deletion events using the payload ref because delete workflows expose the default branch through github.ref.

Coding-Agent: Codex
Codex-Version: codex-cli 0.144.1
Model: gpt-5.6-sol
Reasoning-Effort: xhigh

@wanghan-iapcm wanghan-iapcm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All three changes are correct and the comments explain the non-obvious GitHub Actions behaviors well: switching to ubuntu-latest is required because git-mirror-action is a Docker action (ubuntu-slim has no Docker daemon); the positive branches/tags patterns fix the quirk that branches-ignore alone suppresses tag-triggered runs, and dropping create: avoids duplicate runs since ref creation is delivered as push; and the delete gating correctly inspects github.event.ref (not github.ref, which is the default branch on delete) to suppress only ephemeral gh-readonly-queue deletions while still pruning real refs. Looks good.

@wanghan-iapcm wanghan-iapcm added this pull request to the merge queue Jul 13, 2026
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.

2 participants