Fix linkchecker by remapping root-relative links to learn.microsoft.com#152
Conversation
Add --remap flag to lychee args so root-relative links (e.g. /entra/...) are rewritten to https://learn.microsoft.com/... and validated against the live site instead of failing as unresolvable local paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 303c799: ✅ Validation status: passed
For more details, please refer to the build report. |
The unanchored regex '/(.+)' was matching '/' inside absolute URLs (https://, file:///), producing mangled double-protocol URLs like 'https://https//learn.microsoft.com//github.com/...'. Adding '^' ensures only root-relative links (/entra/...) are remapped to learn.microsoft.com. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit 857a404: ✅ 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 |
Replace the lychee --remap flag with a sed pre-processing step (matching the approach used in MSAL Java) that rewrites root-relative markdown links like [text](/path) to [text](https://learn.microsoft.com/path) before lychee runs. The --remap regex approach couldn't intercept root-relative links before lychee's own root-relative resolver rejected them. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 22d860c: ✅ Validation status: passed
For more details, please refer to the build report. |
Link 'official APIs' to the acquiring-tokens doc, which documents the standard MSAL Python token acquisition methods that the migrate page contrasts with. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 9487468: ✅ Validation status: passed
For more details, please refer to the build report. |
1 participant
Add --remap flag to lychee args so root-relative links (e.g. /entra/...) are rewritten to https://learn.microsoft.com/... and validated against the live site instead of failing as unresolvable local paths.