Skip to content
Merged
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
29 changes: 29 additions & 0 deletions .coderabbit.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
#
# CodeRabbit configuration. Only the settings that differ from CodeRabbit's defaults
# are listed here; everything else stays at the default.
#
# inheritance: without this, a repository .coderabbit.yaml is the sole config source and
# REPLACES any organization/UI settings (the docstring ERROR gate itself lives in the UI,
# since the YAML default is `warning`). Enabling inheritance keeps every UI/org setting β€”
# path filters, request_changes_workflow, custom checks β€” and only overrides the one value
# defined below, so this change stays scoped to docstrings.
inheritance: true
reviews:
pre_merge_checks:
# Docstring Coverage measures a single coverage percentage across a PR's changed files
# and cannot be scoped per-path (the only path mechanism, reviews.path_filters, would
# drop the files from the whole review rather than from just this metric).
#
# This repository holds no Go β€” or any other docstring-bearing β€” production code: it is
# declarative org configuration (deploy/**.yaml), workflow templates, docs and a bash
# test script. So the metric has nothing to measure here and reports 0.00% against an
# 80% threshold, blocking merges on a documentation gap that does not exist (e.g. #103,
# whose only failed pre-merge check was Docstring Coverage 0.00% on a changeset of YAML
# manifests, a bash test and Markdown).
#
# Keep the check as an advisory WARNING β€” CodeRabbit still reports coverage β€” instead of
# a hard merge ERROR. Mirrors devantler-tech/ksail#6145, which made the same change for
# the same un-scopeable-metric reason.
docstrings:
mode: warning
Comment thread
devantler marked this conversation as resolved.
Loading