Skip to content

Conversation

@krystophny
Copy link
Contributor

Summary

  • Remove Fortran zero-config workflow and indirection; use a simple shell bridge for auto mode.

Scope

  • Deleted: src/zero_config/*
  • Updated: src/coverage/workflows/coverage_workflow_core.f90 to call scripts/fpm_coverage_bridge.sh when auto mode is requested (alias: --discover-and-gcov/--gcov).
  • Updated: src/coverage/* to drop zero_config_manager imports and unified discovery to gcov path.
  • Updated: src/config/core/config_defaults_core.f90 to remove unused zero_config_manager use.
  • Added: scripts/fpm_coverage_bridge.sh to run tests (fpm/ctest) and generate .gcov files with gcov.

Verification

  • Baseline and post-change tests with fpm test pass locally (120s timeout enforced). Key outputs include:
    • OK: markdown minimal
    • OK: cli flags minimal and rejections (incl. --zero-config rejected)
    • OK: gcov processing smoke
  • Manual: fortcov --discover-and-gcov now invokes the bridge (non-fatal), then proceeds with gcov discovery and Markdown generation. Manual mode unchanged.

Rationale

  • Simplifies code by removing brittle Fortran zero-config orchestration. Consolidates to a single, explicit auto path via a small shell bridge, aligning with repository direction and reducing maintenance.

@krystophny krystophny merged commit 6cacc14 into main Sep 2, 2025
1 check passed
@krystophny krystophny deleted the fix/issue-1166-remove-zero-config-workflow branch September 2, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants