Skip to content

Update codecov.yml: add ignore patterns for non-code files - #1005

Open
eedri wants to merge 1 commit into
konflux-ci:mainfrom
eedri:update-codecov-ignore-patterns
Open

Update codecov.yml: add ignore patterns for non-code files#1005
eedri wants to merge 1 commit into
konflux-ci:mainfrom
eedri:update-codecov-ignore-patterns

Conversation

@eedri

@eedri eedri commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds ignore patterns to codecov.yml for non-executable files that inflate the coverage denominator:

  • .agents/** — AI agent configurations
  • .claude/** — Claude AI workspace settings
  • .cursor/** — Cursor AI settings
  • skills/** — AI skill definitions
  • docs/** — Documentation files
  • *.md — Markdown files (README, AGENTS.md, etc.)
  • *.toml — TOML configuration files

Existing config (flags, require_ci_to_pass: true) is preserved.

Why

Non-executable files (Markdown, TOML, docs) are counted as "uncovered lines" by Codecov, dragging down patch coverage even though they can't be tested. This was identified as the primary driver of the 13.1% patch coverage drop in the Jul 20-26 weekly report.

🤖 Generated with Claude Code

Adds ignore patterns for AI agent configs, docs, and config files
(.agents/, .claude/, .cursor/, skills/, docs/, *.md, *.toml) to prevent
non-executable files from inflating the coverage denominator.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fullsend-ai-review

fullsend-ai-review Bot commented Aug 5, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 1:40 PM UTC · Completed 1:48 PM UTC
Commit: 701e62a · View workflow run →

@qodo-app-for-konflux-ci

Copy link
Copy Markdown

PR Summary by Qodo

Add Codecov ignore patterns for non-executable docs and AI config files

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Add Codecov ignore globs for docs, markdown, TOML, and AI tooling directories.
• Prevent non-executable files from inflating the coverage denominator and lowering patch coverage.
• Preserve existing Codecov settings (e.g., require_ci_to_pass, flags/status structure).
High-Level Assessment

The following are alternative approaches to this PR:

1. Exclude via coverage tool config (e.g., .coveragerc/pytest)
  • ➕ Keeps the source-of-truth exclusions with the coverage generator
  • ➕ Can prevent excluded files from ever reaching Codecov reports
  • ➖ Tool-specific and language-specific; harder in polyglot repos
  • ➖ May require CI/test runner changes vs. a single Codecov config edit
2. Limit Codecov to specific source roots (path-based include)
  • ➕ More restrictive: only count intended code directories
  • ➕ Avoids maintaining a growing ignore list
  • ➖ Risk of accidentally excluding legitimate code outside the chosen roots
  • ➖ May require repo-specific restructuring or conventions to be reliable

Recommendation: The current Codecov-side ignore list is the simplest, lowest-risk fix for denominator inflation and can be applied immediately without changing CI or test tooling. If the ignore list grows over time, consider moving exclusions upstream into the coverage generator config or switching to an allowlist of code roots to reduce maintenance.

Files changed (1) +9 / -0

Other (1) +9 / -0
codecov.ymlIgnore non-code files in Codecov coverage calculation +9/-0

Ignore non-code files in Codecov coverage calculation

• Adds an 'ignore' section with glob patterns for AI tooling directories, documentation, Markdown, and TOML files. This prevents non-executable files from counting against coverage while preserving existing Codecov settings.

codecov.yml

@qodo-app-for-konflux-ci

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

@fullsend-ai-review

Copy link
Copy Markdown

Review

Findings

Low

  • [missing-authorization] codecov.yml — No linked Jira issue for this coverage configuration change. The AGENTS.md convention states commits should start with a Jira ID. However, this is a one-file configuration-only change to a CI reporting tool that does not affect production code, build artifacts, or runtime behavior. The PR body provides clear rationale (13.1% patch coverage drop from non-executable files).
    Remediation: Consider linking a Jira issue if team process requires it, but this is a minor housekeeping change.

Labels: Small single-file CI config change — minimal review effort

@fullsend-ai-review fullsend-ai-review Bot added ready-for-merge All reviewers approved — ready to merge Review effort 1/5 labels Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant