Skip to content

Conversation

@mxsm
Copy link
Owner

@mxsm mxsm commented Feb 5, 2026

Which Issue(s) This PR Fixes(Closes)

Brief Description

How Did You Test This Change?

Summary by CodeRabbit

  • Chores
    • Modified deployment workflow to refine execution conditions and branch/path filtering.

@rocketmq-rust-robot rocketmq-rust-robot added the documentation📝 Improvements or additions to documentation label Feb 5, 2026
@rocketmq-rust-bot
Copy link
Collaborator

🔊@mxsm 🚀Thanks for your contribution🎉!

💡CodeRabbit(AI) will review your code first🔥!

Note

🚨The code review suggestions from CodeRabbit are to be used as a reference only, and the PR submitter can decide whether to make changes based on their own judgment. Ultimately, the project management personnel will conduct the final code review💥.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

The GitHub Actions deployment workflow is reconfigured to refine trigger conditions: push events now respond to path-specific changes only, pull requests are scoped to the main branch, and deployment steps execute exclusively on non-pull-request events.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/deploy.yml
Removed master branch from push triggers; added path filters for rocketmq-website/** and .github/workflows/deploy.yml; added branches filter for main on pull requests; added conditional check to prevent deployment on pull request events.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A workflow hops to main with care,
Paths filtered here and branches there,
Push deploys, but pull requests stay,
CI/CD flows the rabbit's way! 🚀

🚥 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 directly references issue #6109 and clearly summarizes the main change: updating GitHub Actions deployment configuration for main branch and pull requests, which aligns with the changeset.
Linked Issues check ✅ Passed The PR implements deployment configuration updates as requested in issue #6109, adding path filters, branch conditions, and conditional deployment logic to address main branch and pull request workflows.
Out of Scope Changes check ✅ Passed All changes in the .github/workflows/deploy.yml file are directly related to updating GitHub Actions deployment configuration for main branch and pull requests as specified in issue #6109.
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 doc-6109

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.

@mxsm mxsm merged commit 2e47fc6 into main Feb 5, 2026
17 of 18 checks passed
@rocketmq-rust-bot rocketmq-rust-bot added approved PR has approved and removed ready to review waiting-review waiting review this PR labels Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.65%. Comparing base (3b8d8b6) to head (2aff65f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6110      +/-   ##
==========================================
- Coverage   40.65%   40.65%   -0.01%     
==========================================
  Files         887      887              
  Lines      123251   123251              
==========================================
- Hits        50111    50109       -2     
- Misses      73140    73142       +2     

☔ View full report in Codecov by Sentry.
📢 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.

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

Labels

AI review first Ai review pr first approved PR has approved auto merge documentation📝 Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc📝] Update GitHub Actions deployment configuration for main branch and pull requests

3 participants