Skip to content

Auto-update doc build config#76

Open
alauda-github-idpbot wants to merge 1 commit intomasterfrom
feat/auto-update-doc-build-master
Open

Auto-update doc build config#76
alauda-github-idpbot wants to merge 1 commit intomasterfrom
feat/auto-update-doc-build-master

Conversation

@alauda-github-idpbot
Copy link
Collaborator

@alauda-github-idpbot alauda-github-idpbot commented Feb 12, 2026

auto update doc build config for doc pipeline

Summary by CodeRabbit

  • Chores
    • Documentation build triggering has been updated to exclude tag-based pushes, with builds now only initiated for pushes to main, master, and release-* branches.

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Walkthrough

The pull request modifies the Tekton doc-build pipeline configuration to remove tag-matching logic from the event trigger condition. The disjunctive check for refs/tags/ is eliminated, leaving only branch-matching conditions for main, master, and release branches.

Changes

Cohort / File(s) Summary
Event Trigger Condition Simplification
.tekton/doc-build.yaml
Removed target_branch.matches("refs/tags/") from the CEL expression, narrowing the trigger scope to exclude tag-based pushes while retaining branch pattern matching for main, master, and release branches.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • Auto-update doc build config #46 — Modifies event trigger CEL expressions in the same Tekton pipeline configuration file, with overlapping scope around push/branch condition handling.

Poem

🐰 Tags once danced in the pipeline's embrace,
Now only branches get a build-time place,
Cleaner logic, simpler flow,
Main and release steal the show! 🏗️

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Auto-update doc build config' accurately reflects the main change: a modification to the doc build configuration file (.tekton/doc-build.yaml) to simplify event handling logic.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/auto-update-doc-build-master

No actionable comments were generated in the recent review. 🎉


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 and usage tips.

@cloudflare-workers-and-pages
Copy link

Deploying alauda-servicemesh-s1gzj0 with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3bcffa9
Status: ✅  Deploy successful!
Preview URL: https://7ad8c7cb.alauda-servicemesh-s1gzj0.pages.dev
Branch Preview URL: https://feat-auto-update-doc-build-m.alauda-servicemesh-s1gzj0.pages.dev

View logs

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.

1 participant