pr-definition-of-done

Checklist for merge-ready PRs in coverport. Use when preparing a PR for review or verifying a PR meets all requirements before merging.

Jump to install

Repository: konflux-ci/coverport
Last source activity: July 1, 2026 at 13:44
Detected SKILL.md language: English
Stars: 2
Forks: 11

Install options

The review-first prompt is selected by default. You can switch to a direct command or download a local copy.

Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.

A direct command skips the review prompt. Inspect the source before running it.

npx skills add https://github.com/konflux-ci/coverport --skill pr-definition-of-done

The command stays on one line. Scroll horizontally to inspect it before copying.

Prefer a local copy? Download the files currently available to SkillsMP.

Review the source files

Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.

SKILL.md

Source instructions · Read-only preview

Back to search Back to installation

codecov-onboarding

Onboard repositories to Codecov for unit test coverage tracking. Use this skill when users want to add Codecov integration, configure coverage uploads with flags, or set up CI pipelines to report coverage. Works with GitHub Actions, OpenShift CI (Prow), and GitLab CI. Supports public Codecov (app.codecov.io) and self-hosted Codecov instances.

2026-08-10

codecov-setup

Use when bulk-onboarding multiple repos to Codecov from an audit CSV, or when user says "bulk codecov", "prepare codecov", "enable codecov", "codecov from audit", "onboard repos to codecov", "apply codecov changes", "open the MRs", "push those changes", "enable coverage for all repos", "prepare local for audit.csv". Not for single-repo interactive onboarding (codecov-onboarding) or E2E instrumentation (coverport-integration).

2026-07-15

coverage-audit

Audit a GitHub/GitLab organization's repositories for code coverage status. Generates CSV spreadsheet showing which repos have tests, Codecov integration, and what's missing. Use when user asks to audit an org, create a coverage spreadsheet, or assess coverage gaps across repositories. Also works with pre-existing repo lists (JSON, CSV, text files) spanning multiple orgs/groups.

2026-07-15

View all from this repository

PR Definition of Done

Pre-Submit Checklist

Run pre-commit run --all-files (formatting, trailing whitespace, YAML validation)

All existing tests pass: cd cli && make test

New code has unit tests (table-driven, follow existing patterns)

golangci-lint run ./... passes for modified Go modules (same linters as CI)

Code compiles cleanly: cd cli && go build ./...

If CLI flags changed: update cli/README.md command reference

If new dependency added: justify in PR description (instrumentation must stay zero-dep)

Coverage does not decrease (Codecov patch check is informational, not blocking)

CI Checks That Must Pass

Check

Trigger

Required

CLI Tests (Go 1.24)

All PRs

Yes

Instrumentation Tests (Go matrix)

All PRs

Yes

Lint (golangci-lint + go build)

All PRs

Yes

AGENTS.md line limit

All PRs

Yes

Konflux build (coverport-cli)

PRs touching cli/**

Yes

Codecov coverage

All PRs

Informational

CI Quirks

Konflux Tekton build only triggers when files under cli/ change (path filter in .tekton/coverport-cli-pull-request.yaml). Changes outside cli/ won't get a container build.

Codecov uses OIDC — no token needed, but id-token: write permission must be present.

golangci-lint runs in CI via golangci-lint-action@v9 with v2.12, using per-module .golangci.yml configs.

PR images expire after 5 days — built with tag on-pr-{revision} in RedHat user workloads.

PR Description Guidelines

Explain why the change is needed, not just what changed

Reference any related Jira tickets or GitHub issues

For new CLI commands: include example usage in description

For instrumentation changes: note which languages are affected

Merge Strategy

Squash merge preferred for feature branches

Rebase merge for multi-commit PRs where history matters

Target branch: main

name	pr-definition-of-done
description	Checklist for merge-ready PRs in coverport. Use when preparing a PR for review or verifying a PR meets all requirements before merging.