Skip to content

ci: update e2e-test.yaml/unit-test.yaml#196

Merged
kagol merged 1 commit intodevfrom
kagol/update-test-workflow
Mar 10, 2026
Merged

ci: update e2e-test.yaml/unit-test.yaml#196
kagol merged 1 commit intodevfrom
kagol/update-test-workflow

Conversation

@kagol
Copy link
Member

@kagol kagol commented Mar 10, 2026

PR

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our Commit Message Guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Summary by CodeRabbit

  • Chores
    • Optimized GitHub Actions workflows to run only on relevant template changes, reducing unnecessary pipeline executions while preserving manual trigger options.

@github-actions github-actions bot added the ci Continuous integration label Mar 10, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 10, 2026

Walkthrough

Two GitHub Actions workflows were reconfigured to trigger only on pull requests that modify specific template directories. The e2e test workflow now monitors template/tinyvue/**, while the unit test workflow monitors template/nestJs/**. Both retain manual trigger capabilities via workflow_dispatch.

Changes

Cohort / File(s) Summary
CI/CD Workflow Triggers
.github/workflows/e2e-test.yml, .github/workflows/unit-test.yaml
Narrowed automatic PR triggers to template-specific paths (template/tinyvue/** and template/nestJs/** respectively) while preserving manual workflow_dispatch triggers. Both workflows now avoid running on unrelated changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Workflows now hop with purpose true,
Testing only what is new,
TinyVue and NestJs files lead the way,
No wasteful runs will steal the day! 🚀✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'ci: update e2e-test.yaml/unit-test.yaml' directly references the specific files modified and accurately describes the main change of updating CI workflows.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch kagol/update-test-workflow

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.

@kagol kagol changed the title ci: update *-test.yaml ci: update e2e-test.yaml/unit-test.yaml Mar 10, 2026
@kagol kagol merged commit f70e1c1 into dev Mar 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant