Skip to content

Apply account repository policy - #14

Merged
crmne merged 1 commit into
startfrom
github-automation/account-policy
Oct 13, 2026
Merged

crmne merged 1 commit into
startfrom
github-automation/account-policy

Conversation

@crmne

@crmne crmne commented Oct 13, 2026 •

Copy link
Copy Markdown
Owner

Align this repository with the account-wide maintainer, review, pull request, release-notes, triage, board, sponsorship, and dependency policy.

Summary by CodeRabbit

  • Documentation
    • Added pull request guidance covering review evidence, repository media, dependency sources, and approval requirements.
  • Policy Checks
    • Added automated checks that block pull requests containing reviewer-facing evidence files or dependencies from disallowed sources.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: crmne/ruby_llm-workshop/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c0b6fbfc-e481-490d-838d-81aef5d887df
📥 Commits

Reviewing files that changed from the base of the PR and between 4624f73 and d1f76b6.

📒 Files selected for processing (2)
  • .coderabbit.yaml
  • AGENTS.md

Limit details: You’ve used all 10 included reviews currently available.

📝 Walkthrough

The pull request adds repository guidance and pre-merge checks for review evidence files and dependency sources. The checks allow project-used media and dependencies from package registries or specified maintainer-controlled repositories.

Changes: Pull request policies

Layer / File(s) Summary
Policy guidance and pre-merge checks
AGENTS.md, .coderabbit.yaml
AGENTS.md describes where review evidence belongs and sets dependency-source and approval policies. .coderabbit.yaml adds error-mode checks for review evidence files and dependency sources.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to d1f76

The policy separates dependency-source checks from maintainer approval during review. No identified issue blocks merging after normal checks.

### Architecture Summary

Architecture risk: 🔵 Low · up to d1f76

The change affects 1 system.

Changed systems: AGENTS.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — AGENTS.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in AGENTS.md: Adds pull request policies requiring review evidence to stay in pull request descriptions or comments, limiting repository media to assets used by the project, and restricting dependencies and dependency overrides to registries or maintainer-controlled repositories. Dependency changes require explicit maintainer approval; fixes should be sent upstream and carried in a maintainer-controlled fork until released. Reviewers are instructed to block pull requests that violate either policy.
  • observed — Modified behavior in .coderabbit.yaml: Adds an error-mode pre-merge check that fails when a pull request adds files made to show reviewers the change, while allowing media used by the project itself.
  • observed — Modified behavior in .coderabbit.yaml: Adds an error-mode pre-merge check that fails when a dependency comes from outside its package registry, a repository under github.com/crmne/, or the repository’s own organization; it covers manifests, lockfiles, and package-manager configuration, including Git URLs, forks, patches, overrides, replacements, and other registries. The removed comment described the review workflow and Copilot Triage behavior.
Pre-merge checks · Passed 7
Passed 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 clearly identifies the main change: applying the account repository policy. It is concise and related to the pull request objectives.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Review Evidence Stays Out Of The Repository Passed The pull request changes only .coderabbit.yaml and AGENTS.md. It adds no screenshots, captures, comparison pages, recordings, HTML evidence pages, or other reviewer-evidence files.
Dependencies From Sources The Maintainer Controls Passed The pull request changes only .coderabbit.yaml and AGENTS.md. The changes add policy text and a review check, but do not add or modify a manifest, lockfile, package-manager configuration, git URL,…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 2
  • Waived: $0.50 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.

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

@crmne
crmne merged commit dc3b7c2 into start Oct 13, 2026
6 checks passed
@crmne
crmne deleted the github-automation/account-policy branch October 13, 2026 12:32
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