Skip to content

[#373] Add repository guard to scheduled link-check workflow - #376

Merged
Alomir merged 1 commit into
PecanProject:masterfrom
Labreo:373-link-check-fork-guard
Aug 25, 2026
Merged

[#373] Add repository guard to scheduled link-check workflow#376
Alomir merged 1 commit into
PecanProject:masterfrom
Labreo:373-link-check-fork-guard

Conversation

@Labreo

@Labreo Labreo commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • What: Adds if: github.repository == 'PecanProject/sipnet' to the link-check job in .github/workflows/link-check.yml.
  • Motivation: Prevents the weekly scheduled link-check cron job from running unnecessarily on contributor forks and generating spurious failure notifications.

How was this change tested?

  • Verified YAML syntax and job configuration.
  • pull_request runs targeting PecanProject/sipnet will evaluate to true and continue running normally.

Related issues

Checklist

  • Related issues are listed above.
  • PR title has the issue number in it ([#] <concise description of proposed change>).
  • Tests added/updated for new features (not applicable for workflow configuration).
  • Documentation updated (not applicable).
  • Code formatted with clang-format.

Copilot AI lite review requested due to automatic review settings August 21, 2026 16:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a repository guard to the scheduled “Broken Link Check” GitHub Actions workflow so it only runs in the canonical upstream repository (PecanProject/sipnet), preventing scheduled executions on contributor forks.

Changes:

  • Added if: github.repository == 'PecanProject/sipnet' at the link-check job level in .github/workflows/link-check.yml to skip the job on forks.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Alomir Alomir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, thanks!

@Alomir
Alomir merged commit dbf2756 into PecanProject:master Aug 25, 2026
11 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.

Scheduled link-check workflow runs on contributor forks with no changes

3 participants