Skip to content

chore: add missing entries to the path filter of GitHub actions - #426

Merged
yocaba merged 1 commit into
mainfrom
chore/github-action-triggers
Jul 10, 2026
Merged

chore: add missing entries to the path filter of GitHub actions#426
yocaba merged 1 commit into
mainfrom
chore/github-action-triggers

Conversation

@yocaba

@yocaba yocaba commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

What

Add missing entries to the path filter of GitHub actions

Why

To execute the GitHub actions whenever something related has been changed.

Testing

none

Checklist

  • Tests added/updated n/a
  • No breaking changes (or upgrade path documented above)
  • Readable commit history (squashed and cleaned up as desired)
  • AI code review considered and comments resolved

Summary by CodeRabbit

  • Chores
    • Updated CI triggers so lint and validation checks run when lint configuration or chart-related files change.
    • Adjusted Helm lint workflows to run on the main development branches with relevant path filters.

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Two CI workflow files receive additional path filter entries. golang.yaml adds .golangci.yml to both push and pull_request triggers. helm-lint.yaml adds .github/workflows/helm-lint.yaml to both triggers alongside the existing charts/** filter.

Changes

CI Workflow Trigger Updates

Layer / File(s) Summary
Workflow path filter additions
.github/workflows/golang.yaml, .github/workflows/helm-lint.yaml
golang.yaml adds .golangci.yml to push and pull_request path filters; helm-lint.yaml adds its own workflow file path to both push and pull_request path filters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐇 A bunny checks the lint configs with glee,
Now workflows wake when .golangci shifts free.
Helm charts and workflow files join the watch list too,
No sneaky CI gap shall ever slip through!
Hop hop, the paths are set — all triggers true! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating GitHub Actions path filters.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description matches the template's core sections and states the change, rationale, testing, and checklist, with only optional fields omitted.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/github-action-triggers

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.

@yocaba
yocaba enabled auto-merge June 30, 2026 11:13
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 28440102786

Coverage increased (+0.2%) to 84.052%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 2 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
pkg/controller/workflow_handler.go 2 83.18%

Coverage Stats

Coverage Status
Relevant Lines: 928
Covered Lines: 780
Line Coverage: 84.05%
Coverage Strength: 1293.0 hits per line

💛 - Coveralls

@yocaba

yocaba commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@cbrgm cbrgm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yocaba
yocaba merged commit f96a1a3 into main Jul 10, 2026
23 checks passed
@yocaba
yocaba deleted the chore/github-action-triggers branch July 10, 2026 08:14
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.

4 participants