Skip to content

OpsLevel maturity + ADLC adjustments#7

Merged
chmurph2 merged 4 commits into
masterfrom
cmurphy/configure_claude_code_review
Jun 29, 2026
Merged

OpsLevel maturity + ADLC adjustments#7
chmurph2 merged 4 commits into
masterfrom
cmurphy/configure_claude_code_review

Conversation

@chmurph2

@chmurph2 chmurph2 commented Jun 11, 2026

Copy link
Copy Markdown
Member

Stakeholder Overview (learn more)

OpsLevel maturity (low-hanging fruit) + ADLC adjustments for moqueue. Folds the original automated Claude Code review setup together with the OpsLevel linter fix.

The OpsLevel check flips after this merges to master and OpsLevel re-scans.

Risk Estimate (learn more)

  • ✅ Negligible risk!

CI config only — no library code changes. RuboCop runs out-of-bundle on modern Ruby (this gem's own bundle is ancient) in added-lines, non-blocking mode.

Changes

  • OpsLevel: CI/CD includes linter — adds .github/workflows/rubocop.yml (reviewdog/action-rubocop) + a minimal .rubocop.yml. Added-lines comments on PRs; check run on master pushes.
  • (Name moqueue already satisfies the kebab-case check — single lowercase word, no rename needed. The test suite is ancient RSpec 1.x and is out of scope.)
  • (original) Automated Claude Code PR reviews + actions/checkout v4→v7.

Project Link

Pitch: Premium Blend OpsLevel Maturity — Low-Hanging Fruit

chmurph2 and others added 4 commits June 5, 2026 14:49
- Add 'github.event.pull_request.merged == false' to the pull_request
  trigger guard so the review job is skipped on merged PRs.
- Make cancel-in-progress conditional on github.event_name == 'pull_request'
  so a human @claude comment no longer silently cancels an in-progress
  push-triggered auto-review.
Satisfies the OpsLevel 'CI/CD includes linter' check. Runs RuboCop
out-of-bundle on modern Ruby (this gem's own bundle is ancient) and
reports via reviewdog (added-lines on PRs, check run on master pushes).
@chmurph2 chmurph2 changed the title Configure automated Claude Code PR reviews OpsLevel maturity + ADLC adjustments Jun 26, 2026
@chmurph2 chmurph2 marked this pull request as ready for review June 29, 2026 19:23
@chmurph2 chmurph2 requested a review from a team as a code owner June 29, 2026 19:24
@chmurph2

Copy link
Copy Markdown
Member Author

Ready for review.

@chmurph2 chmurph2 merged commit 632834d into master Jun 29, 2026
5 checks passed
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