aRustyDev / aRustyDev/pre-commit-hooks

track: Code coverage reporting implementation

Open
#29 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
5
Forks
3
PR merge metrics
No merged PRs in 30d

Description

## Overview
Track the implementation and improvement of code coverage reporting for all hook scripts.

## Current Status
- ✅ Coverage workflow created (`.github/workflows/coverage.yml`)
- ✅ kcov integration configured
- ⚠️ Coverage currently at 0% (no hooks passing tests)

## Goals
- [ ] Fix hook scripts to pass shellcheck
- [ ] Achieve 50% coverage
- [ ] Achieve 80% coverage target
- [ ] Add coverage badge to README
- [ ] Integrate coverage checks in PR reviews

## Tracking Metrics
- Current coverage: 0%
- Target coverage: 80%
- Number of hooks tested: 3/26
- Number of test files: 6

## Related PRs
- Initial coverage setup: (this commit)

## Dependencies
- Depends on #27 (test coverage for all hooks)
- Blocks v1.0.0 release

## Action Items
1. Fix shellcheck errors in hook scripts
2. Update tests to handle hook script issues
3. Add coverage badge to README
4. Configure coverage as required PR check

## Milestone
Test Coverage 80%

Contributor guide

Open the contributing guide

Research direction

Start by reviewing .github/workflows/coverage.yml, the 6 existing test files, and the dependency issue #27. Check the 26 hook scripts and current 3/26 test coverage, then address the listed shellcheck and test issues. Done means reaching the stated coverage targets, adding a README badge, and configuring coverage as a required PR check.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, shell
Domain
ci-cd, release, testing
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.