Skip to content

Fix linkchecker: add config file and exclude live branch#250

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

Fix linkchecker: add config file and exclude live branch#250
Dickson-Mwendia merged 1 commit into
mainfrom
link-checker-issue

Conversation

@Dickson-Mwendia
Copy link
Copy Markdown
Collaborator

  • Add .lychee.toml so lychee doesn't mistakenly parse package.json as config
  • Pass --config=.lychee.toml explicitly in workflow args
  • Remove pull_request trigger (push trigger already covers PR branches)
  • Exclude live branch from push triggers (already validated on main)

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

Learn Build status updates of commit 60ac153:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/linkchecker.yml ✅Succeeded
.lychee.toml ✅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.

- Add .lychee.toml so lychee doesn't mistakenly parse package.json as config
- Pass --config=.lychee.toml explicitly in workflow args
- Remove pull_request trigger (push trigger already covers PR branches)
- Exclude live branch from push triggers (already validated on main)

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

Learn Build status updates of commit d3e148e:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/linkchecker.yml ✅Succeeded
.lychee.toml ✅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.

@Dickson-Mwendia Dickson-Mwendia merged commit add169c into main May 19, 2026
5 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