Skip to content

fix(ci): add docs-only paths-ignore to container and coverage workflows#1762

Merged
josecelano merged 1 commit into
torrust:developfrom
josecelano:fix/1743-docs-only-ci-fast-path-missing-workflows
May 12, 2026
Merged

fix(ci): add docs-only paths-ignore to container and coverage workflows#1762
josecelano merged 1 commit into
torrust:developfrom
josecelano:fix/1743-docs-only-ci-fast-path-missing-workflows

Conversation

@josecelano
Copy link
Copy Markdown
Member

Bug

Issue #1743 added paths-ignore rules to the CI "docs-only fast path" for
testing.yaml and os-compatibility.yaml, but two workflows were missed:

  • container.yaml — the container build job
  • generate_coverage_pr.yaml — the coverage report job

As a result, docs-only pull requests (e.g. PR #1760) were still triggering the
container build and coverage jobs unnecessarily, wasting CI minutes and
delaying feedback.

Fix

Add the same paths-ignore exclusions to both workflows:

paths-ignore:
  - "**/*.md"
  - "project-words.txt"

Also update the path-policy comment in docs-lint.yaml to accurately list all
workflows that carry the paths-ignore rule (replacing the stale list that
mentioned db-compatibility.yaml and db-benchmarking.yaml, which are already
scoped to code paths via paths: inclusion rules).

Changed files

  • .github/workflows/container.yaml — Added paths-ignore to both push and pull_request triggers, plus path-policy comment.
  • .github/workflows/generate_coverage_pr.yaml — Added paths-ignore to the pull_request trigger, plus path-policy comment.
  • .github/workflows/docs-lint.yaml — Updated header comment to correctly list workflows that carry paths-ignore rules.

References

@josecelano josecelano requested a review from a team as a code owner May 12, 2026 10:53
@josecelano josecelano self-assigned this May 12, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.44%. Comparing base (7561d58) to head (2b177a2).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1762   +/-   ##
========================================
  Coverage    79.44%   79.44%           
========================================
  Files          369      369           
  Lines        27328    27328           
  Branches     27328    27328           
========================================
  Hits         21710    21710           
  Misses        5337     5337           
  Partials       281      281           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The issue torrust#1743 implementation added paths-ignore rules to testing.yaml
and os-compatibility.yaml but missed container.yaml and
generate_coverage_pr.yaml. Docs-only pull requests (e.g. PR torrust#1760) were
still triggering the container build and coverage jobs unnecessarily.

Add the same paths-ignore exclusions to both workflows and update the
docs-lint.yaml path-policy comment to accurately reflect which workflows
carry the rule.
@josecelano josecelano force-pushed the fix/1743-docs-only-ci-fast-path-missing-workflows branch from 8b748b2 to 2b177a2 Compare May 12, 2026 11:18
@josecelano
Copy link
Copy Markdown
Member Author

ACK 2b177a2

@josecelano josecelano merged commit c523b1d into torrust:develop May 12, 2026
10 of 13 checks passed
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.

1 participant