Closed krapie closed 3 weeks ago
The CI workflow in GitHub Actions has been enhanced with the addition of a new job, ci-target-check
, which determines the execution of subsequent jobs based on file path changes. This improvement makes the CI process more efficient by conditionally running the build
, bench
, and sharding-test
jobs only when relevant changes are detected. The workflow also expands the scope of monitored paths and aligns naming conventions for consistency.
Files | Change Summary |
---|---|
.github/workflows/ci.yml |
Added ci-target-check job; modified build , bench , and sharding-test jobs to depend on it; expanded ignored paths to include design/** ; renamed sharding_test to sharding-test . |
🐇 In the garden of code where changes bloom,
A check now ensures we make less room.
With paths filtered, efficient we glide,
Only the needed will run side by side.
Hooray for the workflow, let progress be bright,
As we hop through our tasks, all feels just right! 🌼✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 51.06%. Comparing base (
80c6ea0
) to head (37a2401
). Report is 2 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What this PR does / why we need it:
Revised version of fine-tuned CI workflows in PR https://github.com/yorkie-team/yorkie/pull/964, which is
actions/checkout
fordorny/paths-filter
.The CI failed in
main
branch, due to dorny/paths-filter action's behavior. It does not require checkout when triggered by PR, but it requires checkout on other triggers.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation:
Checklist:
Summary by CodeRabbit