Skip to content

ci: run lint and tests on all branch pushes - #99

Merged
Daniel Willim (kognic-willim) merged 4 commits into
masterfrom
ci-test-all-branch-pushes
Aug 18, 2026
Merged

ci: run lint and tests on all branch pushes#99
Daniel Willim (kognic-willim) merged 4 commits into
masterfrom
ci-test-all-branch-pushes

Conversation

@kognic-willim

@kognic-willim Daniel Willim (kognic-willim) commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

The "*" glob in the push trigger does not match /, so pushes to renovate/* and dependabot/* branches never ran lint or tests — bot dependency PRs only got CodeQL. Changing to "**" makes the workflow run on every branch push.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated automated workflow triggers to run for pushes to branches with any name.

The "*" glob does not match "/", so pushes to renovate/* and
dependabot/* branches never triggered the workflow, leaving bot PRs
untested.
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ec85a983-66f8-422b-881f-67b833639329

📥 Commits

Reviewing files that changed from the base of the PR and between db5eaa4 and 5f1f64e.

📒 Files selected for processing (1)
  • .github/workflows/python-package.yml

Included review availability: Your plan includes up to 3 reviews per rolling hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The Python package workflow now runs for pushes to branches at any path depth by using the ** branch pattern.

Changes

Python package workflow

Layer / File(s) Summary
Recursive push trigger
.github/workflows/python-package.yml
The push trigger branch pattern changed from * to **, matching branches with nested path segments.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 5f1f6

This localized workflow change expands lint and test execution to all branch pushes, including dependency-update branches, with no actionable merge-blocking risk remaining beyond normal checks.

Suggested reviewers: jonaslohsekog

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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 Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the CI change that runs lint and tests on pushes to all branches.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci-test-all-branch-pushes
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch ci-test-all-branch-pushes

Comment @coderabbitai help to get the list of available commands.

@kognic-willim
Daniel Willim (kognic-willim) marked this pull request as ready for review August 18, 2026 11:59
@kognic-willim
Daniel Willim (kognic-willim) requested a review from a team as a code owner August 18, 2026 11:59
@kognic-willim
Daniel Willim (kognic-willim) merged commit b072f98 into master Aug 18, 2026
10 checks passed
@kognic-willim
Daniel Willim (kognic-willim) deleted the ci-test-all-branch-pushes branch August 18, 2026 12:31
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.

2 participants