Fix linkChecker: transform root-relative links, add PR trigger for live branch#25
Conversation
|
Learn Build status updates of commit 145db12: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Dickson-Mwendia
force-pushed
the
fix-link-checker-issue
branch
from
145db12 to
a915c87
Compare
Dickson-Mwendia
changed the title
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit a915c87: ✅ Validation status: passed
For more details, please refer to the build report. |
Dickson-Mwendia
force-pushed
the
fix-link-checker-issue
branch
from
a915c87 to
7b648d8
Compare
|
Learn Build status updates of commit 7b648d8: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Dickson-Mwendia
force-pushed
the
fix-link-checker-issue
branch
from
7b648d8 to
5833823
Compare
|
Learn Build status updates of commit 5833823: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
- Add sed step to convert root-relative markdown links to full learn.microsoft.com URLs before lychee runs (matches Java repo pattern) - Remove --remap approach (lychee treats root-relative as filesystem paths) - Add branches-ignore: [live] on push trigger - Add pull_request trigger for all branches - Fix broken v1.3.0-preview release link (tag never existed, use v1.3.1) - Fix .lycheeignore: use regex pattern for localhost, add aka.ms/region-map Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Dickson-Mwendia
force-pushed
the
fix-link-checker-issue
branch
from
5833823 to
219d184
Compare
|
Learn Build status updates of commit 219d184: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
1 participant
This pull request introduces several improvements to link validation in the CI workflow and updates documentation for managed identity support in MSAL for Go. The most significant changes are grouped below by theme.
CI Workflow Improvements:
.github/workflows/linkchecker.ymlworkflow is updated to run on pull requests in addition to pushes, and now ignores thelivebranch for push events./path) to absolute URLs (https://learn.microsoft.com/path) before running the link checker, ensuring that such links are properly validated.Link Checker Configuration:
.lycheeignorefile is updated to ignore all URLs matchinghttp://localhost.*and to addhttps://aka.ms/region-mapto the ignore list, improving link checker accuracy and reducing false positives.Documentation Update:
msal-go/advanced/managed-identity.mddocumentation is updated to state that managed identity support is available starting with MSAL for Go version1.3.1(previously1.3.0-preview).