Skip to content

ci: run link analysis for changed CSV data - #3599

Open
bcabreraike-cmyk wants to merge 1 commit into
Chain-Love:mainfrom
bcabreraike-cmyk:codex/auto-link-check-csv-prs
Open

ci: run link analysis for changed CSV data#3599
bcabreraike-cmyk wants to merge 1 commit into
Chain-Love:mainfrom
bcabreraike-cmyk:codex/auto-link-check-csv-prs

Conversation

@bcabreraike-cmyk

Copy link
Copy Markdown
Contributor

Summary

Run link analysis automatically when a pull request changes link-bearing CSV data, while preserving manual triggering with check-links and workflow_dispatch.

The workflow now compares extracted URLs with the base branch and checks only URLs introduced or changed by the pull request. Historical URLs on otherwise touched rows no longer create unrelated review noise.

Type of change

  • CI/workflow change only (no data or schema change)

Scope

  • File: .github/workflows/link-check-analysis.yaml
  • No provider, offer, network, or listing data changed.
  • The automatic path is limited to CSVs under listings/ or references/, plus edits to this workflow so it validates itself.

Validation evidence

Security

  • Workflow permissions remain read-only (contents: read).
  • Checkout credentials are not persisted.
  • No secrets, write tokens, or external credentials are used.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 0
🔗 Unique 0
✅ Successful 0
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 0
🔗 Unique 0
✅ Successful 0
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@USS-Supervisor USS-Supervisor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE
Risk: LOW
Summary: The workflow change is narrowly scoped to Link Check (analysis) and keeps read-only permissions while making link analysis run for CSV/link-check workflow changes. The diff checks only changed link-bearing data against the base branch, which addresses the recurring inherited-link noise without changing repository data.

Findings: None.

Confidence: HIGH

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