Skip to content

Fix build crash caused by broken symlinks#46

Merged
shenxianpeng merged 4 commits into
mainfrom
fix/3785-broken-symlink-crash
Apr 26, 2026
Merged

Fix build crash caused by broken symlinks#46
shenxianpeng merged 4 commits into
mainfrom
fix/3785-broken-symlink-crash

Conversation

@shenxianpeng
Copy link
Copy Markdown
Member

@shenxianpeng shenxianpeng commented Apr 26, 2026

When a dangling symlink exists in the docs directory, shutil.copyfile raises an exception that crashes the entire build. Check for broken symlinks before copying, log a warning, and skip the file instead of crashing.

Closes mkdocs/mkdocs#3785

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup
  • CI / build / dependency update
  • Other (describe below)

Checklist

  • New tests added for new behavior (if applicable)
  • Documentation updated (if applicable)
  • Release notes docs/about/release-notes.md updated (if applicable)

When a dangling symlink exists in the docs directory, shutil.copyfile
raises an exception that crashes the entire build. Check for broken
symlinks before copying, log a warning, and skip the file instead of
crashing.

Closes mkdocs/mkdocs#3785
@shenxianpeng shenxianpeng changed the title Fix build crash caused by broken symlinks (#3785) Fix build crash caused by broken symlinks Apr 26, 2026
@github-actions github-actions Bot added the bug Something isn't working label Apr 26, 2026
Comment thread docs/about/release-notes.md Outdated
@shenxianpeng shenxianpeng merged commit 83d3109 into main Apr 26, 2026
23 checks passed
@shenxianpeng shenxianpeng deleted the fix/3785-broken-symlink-crash branch April 26, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant