Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -111,13 +111,6 @@ jobs:
run: |
nix develop --command cargo tarpaulin --out xml --all-features --verbose

- name: Upload Coverage
uses: codecov/codecov-action@v4
with:
files: ./cobertura.xml
fail_ci_if_error: false
continue-on-error: true

# MSRV check (use specific Rust version, can't do this easily in Nix)
# Only run on main branch pushes, scheduled runs, and manual triggers - not every PR
msrv:
Expand Down
50 changes: 1 addition & 49 deletions .github/workflows/claude-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -248,52 +248,4 @@ jobs:
echo "✅ Auto-merge enabled! PR will merge automatically when:"
echo " - All required checks pass (test, lint, build)"
echo " - All conversations are resolved"
echo " - Branch is up to date with main"

require-claude-for-main:
name: Require Claude Review for Main
runs-on: ubuntu-latest
if: github.event_name == 'pull_request' && github.event.pull_request.base.ref == 'main'
steps:
- name: Check for Claude review
uses: actions/github-script@v7
with:
script: |
const pr_number = context.payload.pull_request.number;

// Check if Claude review has been completed by looking for:
// 1. Comments with automated checks from Claude review workflow
// 2. Actual PR reviews (APPROVED or COMMENT)
const comments = await github.rest.issues.listComments({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: pr_number
});

const reviews = await github.rest.pulls.listReviews({
owner: context.repo.owner,
repo: context.repo.repo,
pull_number: pr_number
});

// Check for automated checks comment (posted by Claude Auto Review job)
const has_automated_checks = comments.data.some(comment =>
comment.body.includes('🔍 Automated Checks') &&
comment.body.includes('Claude is reviewing')
);

// Check for actual review with Claude AI approval/feedback
const has_review = reviews.data.some(review =>
review.body && (
review.body.includes('Claude AI approved') ||
review.body.includes('Claude AI review')
)
);

const has_claude_review = has_automated_checks && has_review;

if (!has_claude_review) {
core.setFailed('⚠️ Claude review required for merging to main. Comment "/claude review" to trigger.');
} else {
console.log('✅ Claude review found');
}
echo " - Branch is up to date with main"
2 changes: 1 addition & 1 deletion .github/workflows/renovate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ jobs:
continue-on-error: true

- name: Self-hosted Renovate
uses: renovatebot/github-action@v40.3.12
uses: renovatebot/github-action@v44.0.2
env:
# Use GitHub App token if available, otherwise use GITHUB_TOKEN
RENOVATE_TOKEN: ${{ steps.get_token.outputs.token || secrets.GITHUB_TOKEN }}
Expand Down
Loading