Skip to content

ci(links): avoid duplicate broken link reports - #2073

Merged
Lee-W merged 1 commit into
masterfrom
bearomorphism-dedupe-broken-link-reports
Aug 21, 2026
Merged

ci(links): avoid duplicate broken link reports#2073
Lee-W merged 1 commit into
masterfrom
bearomorphism-dedupe-broken-link-reports

Conversation

@bearomorphism

Copy link
Copy Markdown
Collaborator

Description

Prevent the scheduled link checker from opening repeated Broken Links Report issues while an exact-title report is already open. The workflow paginates open issues, excludes pull requests, and creates a report only when no matching issue exists.

Checklist

Was generative AI tooling used to co-author this PR?

  • Yes (GitHub Copilot, powered by GPT-5.6 Sol)

Generated-by: GitHub Copilot following the guidelines

Code Changes

  • Add test cases to all the changes you introduce
  • Run uv run poe all locally to ensure this change passes linter check and tests
  • Manually test the changes:
    • Verify the feature/bug fix works as expected in real-world scenarios
    • Test edge cases and error conditions
    • Ensure backward compatibility is maintained
    • Document any manual testing steps performed
  • Update the documentation for the changes

Documentation Changes

N/A

  • Run uv run poe doc locally to ensure the documentation pages renders correctly
  • Check and fix any broken links (internal or external)

Expected Behavior

When a scheduled check finds broken links, it skips issue creation if an open issue titled Broken Links Report already exists. A matching pull request does not suppress report creation.

Steps to Test This Pull Request

  1. Run the report script logic with an open issue titled Broken Links Report and confirm issue creation is skipped.
  2. Run it with no matching issue, and with a matching pull request, and confirm a report issue is created.
  3. Run uv run prek run check-yaml --files .github/workflows/links.yml.

Additional Context

The workflow create/skip branches were exercised with mocked GitHub API responses.

Fixes: #2071

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.26%. Comparing base (d910b66) to head (e539aa7).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2073   +/-   ##
=======================================
  Coverage   98.26%   98.26%           
=======================================
  Files          61       61           
  Lines        2829     2829           
=======================================
  Hits         2780     2780           
  Misses         49       49           

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

@bearomorphism
bearomorphism marked this pull request as ready for review August 20, 2026 15:09
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Commitizen bump preview

No commits in this PR are eligible for a version bump.

@bearomorphism

bearomorphism commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator Author

I noticed there are ~20 broken link report issues when I fixed #2071. If there is a broken link, one issue is enough I think.

@Lee-W
Lee-W merged commit d914b4d into master Aug 21, 2026
22 checks passed
@Lee-W
Lee-W deleted the bearomorphism-dedupe-broken-link-reports branch August 21, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Broken Links Report

2 participants