Skip to content

chore(ci): auto-merge dependabot PRs#86

Merged
janisz merged 5 commits intomainfrom
auto-merge
Mar 30, 2026
Merged

chore(ci): auto-merge dependabot PRs#86
janisz merged 5 commits intomainfrom
auto-merge

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Mar 23, 2026

Description

Auto-merge dependabot PRs but skip e2e tests auto updates as mcp checker will be green even when not working.

Validation

Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.88%. Comparing base (0c425b5) to head (cd426f0).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   73.88%   73.88%           
=======================================
  Files          30       30           
  Lines        1302     1302           
=======================================
  Hits          962      962           
  Misses        300      300           
  Partials       40       40           
Flag Coverage Δ
integration 73.88% <ø> (ø)
unit 73.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
@janisz janisz requested a review from mtodor March 23, 2026 12:26
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 23, 2026

E2E Test Results

Commit: cd426f0
Workflow Run: View Details
Artifacts: Download test results & logs

=== Evaluation Summary ===

  ✓ list-clusters (assertions: 3/3)
  ✓ cve-detected-workloads (assertions: 3/3)
  ✓ cve-detected-clusters (assertions: 3/3)
  ~ cve-nonexistent (assertions: 2/3)
      - MaxToolCalls: Too many tool calls: expected <= 5, got 7
  ✓ cve-cluster-does-exist (assertions: 3/3)
  ~ cve-cluster-does-not-exist (assertions: 2/3)
      - ToolsUsed: Required tool not called: server=stackrox-mcp, tool=, pattern=list_clusters
  ✓ cve-clusters-general (assertions: 3/3)
  ✓ cve-cluster-list (assertions: 3/3)
  ✓ cve-log4shell (assertions: 3/3)
  ✓ cve-multiple (assertions: 3/3)
  ✓ rhsa-not-supported (assertions: 2/2)

Tasks:      11/11 passed (100.00%)
Assertions: 30/32 passed (93.75%)
Tokens:     ~55523 (estimate - excludes system prompt & cache)
MCP schemas: ~12738 (included in token total)
Agent used tokens:
  Input:  15011 tokens
  Output: 21824 tokens
Judge used tokens:
  Input:  62753 tokens
  Output: 58983 tokens

Copy link
Copy Markdown
Collaborator

@mtodor mtodor left a comment

Choose a reason for hiding this comment

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

What do you think to add label for directory: "/" and also require that label to be set in order to trigger auto merge?

In that case we will get PRs for both (base go.mod and tools go.mod), but only PRs updating base modules will be auto merged.

Add label-based control for Dependabot auto-merge instead of excluding
entire modules. Root module dependencies get auto-merge label, while
e2e-tests/tools dependencies require manual review.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@janisz janisz requested a review from mtodor March 27, 2026 16:54
@janisz janisz merged commit f0d2054 into main Mar 30, 2026
8 checks passed
@janisz janisz deleted the auto-merge branch March 30, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants