Skip to content

ci: separate test deps from prod in dependabot config#136

Open
caxu-rh wants to merge 1 commit into
redhat-openshift-ecosystem:mainfrom
caxu-rh:gomod-dep-grouping
Open

ci: separate test deps from prod in dependabot config#136
caxu-rh wants to merge 1 commit into
redhat-openshift-ecosystem:mainfrom
caxu-rh:gomod-dep-grouping

Conversation

@caxu-rh

@caxu-rh caxu-rh commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Similar to redhat-openshift-ecosystem/openshift-preflight@61a14e8.

Summary by CodeRabbit

  • Chores
    • Updated dependency update rules for Go modules to better separate test-related packages from production dependencies.
    • Adjusted package grouping so Ginkgo/Gomega updates are handled as test dependencies and excluded from production dependency grouping.

Signed-off-by: Caleb Xu <caxu@redhat.com>
@openshift-ci openshift-ci Bot requested review from acornett21 and komish June 24, 2026 18:49
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 49381450-0137-442b-b54d-8a7f78a420ea

📥 Commits

Reviewing files that changed from the base of the PR and between b06d9eb and 951069e.

📒 Files selected for processing (1)
  • .github/dependabot.yml

Walkthrough

.github/dependabot.yml updates Go module grouping rules to classify Ginkgo and Gomega packages as test dependencies and exclude them from production dependency grouping.

Changes

Dependabot Go dependency grouping

Layer / File(s) Summary
Go dependency group patterns
.github/dependabot.yml
The Dependabot groups configuration replaces dependency-type-based dev/prod entries with test-dependencies patterns for github.com/onsi/ginkgo* and github.com/onsi/gomega*, plus a prod-dependencies exclusion list for those same packages.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 7
✅ Passed checks (7 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the Dependabot config change to separate test dependencies from production dependencies.
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.
Stable And Deterministic Test Names ✅ Passed No Ginkgo titles use dynamic data; the only test titles are static literals in version/version_test.go, and this PR only changes dependabot config.
Test Structure And Quality ✅ Passed PR only changes .github/dependabot.yml; no Ginkgo test code was modified, so this test-quality check is not applicable.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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.

1 participant