ci: improve workflows#15037
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughGitHub Actions workflows are updated across the repository to add concurrency policies with cancel-in-progress for pull requests, set ChangesCI Workflow Updates
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Size ReportBundles
Usages
|
@vue/compiler-core
@vue/compiler-dom
@vue/compiler-sfc
@vue/compiler-ssr
@vue/reactivity
@vue/runtime-core
@vue/runtime-dom
@vue/server-renderer
@vue/shared
vue
@vue/compat
commit: |
Description
Hi, there are a few improvements that can be made to the workflows.
ubuntu-slimrunnerUse
ubuntu-slimforclose-cant-reproduce-issues.yml,lock-closed-issues.yml,ecosystem-ci-trigger.yml, andsize-report.yml. The runner readme provides more info about the tools and the blog post gives some potential use cases.persist-credentials: falseUse
persist-credentials: falseon everyactions/checkout(none of the jobs perform subsequent git operations —pnpm release --publishOnlyinrelease.ymldoesn't touch git). See usage for more info.parallelkeywordUse
parallelfor independent steps intest.ymlandsize-report.ymlGitHub Actions announced the
parallelkeyword — syntactic sugar that groups independent steps to run concurrently within a single job, then waits for all of them before continuing. See the workflow syntax docs for the full reference.Concurrency
Add
concurrencyto the PR-triggered workflows so a new push cancels the previous in-flight run of the same workflow.ci.ymlandsize-data.ymluse the|| github.reffallback to keep per-branch grouping forpushevents, while still cancelling on PRs.Related
Summary by CodeRabbit
Bug Fixes
Chores