Skip to content

CI tooling fixes and docs cleanup - #17

Merged
KARTIKrocks merged 2 commits into
mainfrom
chore/ci-tooling-and-docs
Jul 8, 2026
Merged

KARTIKrocks merged 2 commits into
mainfrom
chore/ci-tooling-and-docs

Conversation

@KARTIKrocks

Copy link
Copy Markdown
Owner

CI:

  • Lint every module (matrix over root + 6 integrations + 2 parsers); previously only the root module was linted, so satellite lint issues slipped through.
  • Align golangci-lint to v2.12.2 (matches the Makefile pin).

Dependabot:

  • Group each satellite module's gomod updates (and github-actions) into a single PR per module to cut update noise.

golangci-lint:

  • Drop the errcheck exclusion for the non-existent examples/ dir.

Docs:

  • Rename CLAUDE.md to AGENTS.md (tool-agnostic agent guidance) and update all references (CONTRIBUTING, PR template, .coderabbit.yaml).
  • Remove stale references to the internal PRODUCTION_READINESS roadmap from CONTRIBUTING.

Summary

What does this PR change, and why?

Closes #

Type of change

  • Bug fix
  • New detection rule
  • New integration / parser
  • Feature / enhancement
  • Docs only
  • Refactor / chore

Checklist

  • make ci passes (fmt-check, vet, lint, test-race) across all modules
  • Added/updated tests (and, where practical, a failure-mode check)
  • Updated docs as needed (README.md, CLAUDE.md, .sqlguard.example.yml)
  • Added an entry under ## [Unreleased] in CHANGELOG.md
  • No new third-party deps in analyzer / middleware / reporter
  • Findings stay redaction-safe (no raw literals leak into a Result)

Notes for reviewers

Anything reviewers should focus on — tricky areas, trade-offs, follow-ups.

CI:
- Lint every module (matrix over root + 6 integrations + 2 parsers);
  previously only the root module was linted, so satellite lint issues
  slipped through.
- Align golangci-lint to v2.12.2 (matches the Makefile pin).

Dependabot:
- Group each satellite module's gomod updates (and github-actions) into
  a single PR per module to cut update noise.

golangci-lint:
- Drop the errcheck exclusion for the non-existent examples/ dir.

Docs:
- Rename CLAUDE.md to AGENTS.md (tool-agnostic agent guidance) and update
  all references (CONTRIBUTING, PR template, .coderabbit.yaml).
- Remove stale references to the internal PRODUCTION_READINESS roadmap
  from CONTRIBUTING.
@coderabbitai

coderabbitai Bot commented Jul 8, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@KARTIKrocks, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 48 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: af3df3a2-cb87-4bf1-8f06-f612727b7108

📥 Commits

Reviewing files that changed from the base of the PR and between 361e007 and 0954209.

📒 Files selected for processing (7)
  • .coderabbit.yaml
  • .github/PULL_REQUEST_TEMPLATE.md
  • .github/dependabot.yml
  • .github/workflows/ci.yml
  • .golangci.yml
  • AGENTS.md
  • CONTRIBUTING.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/ci-tooling-and-docs

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.

@KARTIKrocks
KARTIKrocks merged commit f012b74 into main Jul 8, 2026
15 checks passed
@KARTIKrocks
KARTIKrocks deleted the chore/ci-tooling-and-docs branch July 8, 2026 16:50
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