Skip to content

Add pull_request trigger to linkchecker workflow#248

Merged
Dickson-Mwendia merged 1 commit into
mainfrom
link-checker-issue
May 19, 2026
Merged

Add pull_request trigger to linkchecker workflow#248
Dickson-Mwendia merged 1 commit into
mainfrom
link-checker-issue

Conversation

@Dickson-Mwendia
Copy link
Copy Markdown
Collaborator

The linkchecker workflow was missing a pull_request trigger, so it never ran as a PR check. This adds the trigger so broken links are caught before merging.

@Dickson-Mwendia Dickson-Mwendia requested a review from jmprieur as a code owner May 19, 2026 07:37
@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit b5e2fe9:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/linkchecker.yml ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

The linkchecker workflow was missing a pull_request trigger, so it never
ran as a PR check. This adds the trigger so broken links are caught
before merging. Also excludes the live branch from push triggers since
links are already validated on main.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod
Copy link
Copy Markdown

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown

Learn Build status updates of commit 4404b52:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/linkchecker.yml ✅Succeeded

For more details, please refer to the build report.

@Dickson-Mwendia Dickson-Mwendia enabled auto-merge May 19, 2026 07:44
@Dickson-Mwendia Dickson-Mwendia disabled auto-merge May 19, 2026 07:44
@Dickson-Mwendia Dickson-Mwendia merged commit 409790b into main May 19, 2026
3 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