Skip to content

[Sync] Update project files from source repository (d8c8cc4)#105

Merged
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260130-161328-d8c8cc4
Jan 30, 2026
Merged

[Sync] Update project files from source repository (d8c8cc4)#105
mrz1836 merged 1 commit intomasterfrom
chore/sync-files-mrz-tools-20260130-161328-d8c8cc4

Conversation

@mrz1836
Copy link
Owner

@mrz1836 mrz1836 commented Jan 30, 2026

What Changed

  • Added a new step "🎯 Enforce Coverage Threshold" to the fortress-coverage workflow that validates test coverage against the configured threshold
  • The step locates the coverage file by searching three possible locations: coverage-artifacts/coverage-data/, coverage-artifacts/, and the repository root
  • Uses the go-coverage parse command with --threshold flag to enforce minimum coverage requirements
  • Fails the build with exit code if coverage drops below GO_COVERAGE_THRESHOLD, providing detailed error messages and remediation guidance
  • The enforcement step only runs when GO_COVERAGE_THRESHOLD is set and not equal to '0' or '0.0'
  • Includes informative output showing which coverage exclusions were applied during test generation (paths, files, tests, and generated code)

Why It Was Necessary

  • Ensures that code coverage standards are automatically enforced during CI builds rather than relying on manual review
  • Provides immediate feedback to developers when test coverage falls below acceptable thresholds
  • Prevents regression in test coverage by failing builds that don't meet the minimum quality bar

Testing Performed

  • The change adds enforcement logic that will be tested when the workflow runs with coverage data
  • The step includes robust error handling for missing coverage files with clear diagnostic output
  • Conditional logic ensures the threshold check only runs when explicitly configured

Impact / Risk

  • Breaking Change: Builds may start failing if existing coverage is below the configured threshold
  • Risk: Low - the enforcement is optional and only active when GO_COVERAGE_THRESHOLD is explicitly set to a non-zero value
  • Developer Experience: Improved clarity on coverage requirements with actionable error messages explaining how to fix threshold failures

@mrz1836 mrz1836 self-assigned this Jan 30, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Jan 30, 2026
@github-actions github-actions bot added size/M Medium change (51–200 lines) update General updates labels Jan 30, 2026
@github-actions
Copy link
Contributor

Code Coverage Analysis

🟡 Overall Coverage: 85.2%
No Go files modified in this PR

Project coverage remains at 85.2% (3.1K/3.7K statements)

Changes: 1 config file

Coverage Metrics

Metric Value Grade Trend
Percentage 85.2% ⭐ B+ 📊
Statements 3.1K/3.7K ⭐ B+ No change
Quality Score 85.2/100 ⭐ B+ 📈

Coverage Breakdown

████████████░░░ 85.2%

Resources


Generated via go-coverage2026-01-30 21:18:47 UTC

github-actions bot added a commit that referenced this pull request Jan 30, 2026
🏷️ Generated with GoFortress
@mrz1836
Copy link
Owner Author

mrz1836 commented Jan 30, 2026

LGTM!

@mrz1836 mrz1836 merged commit 9b0710f into master Jan 30, 2026
44 checks passed
@github-actions github-actions bot deleted the chore/sync-files-mrz-tools-20260130-161328-d8c8cc4 branch January 30, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/M Medium change (51–200 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant