Skip to content

ci: validate learning note links - #1

Open
DorianChn wants to merge 2 commits into
mainfrom
codex/validate-note-links-20260914
Open

DorianChn wants to merge 2 commits into
mainfrom
codex/validate-note-links-20260914

Conversation

@DorianChn

Copy link
Copy Markdown
Owner

Summary

  • add a standard-library validator for all Markdown learning notes
  • reject empty notes and broken or repository-escaping relative links
  • run the validator in CI and document the local command

Validation

  • uv run --no-project python scripts/validate_notes.py
  • validated 2 Markdown files and all relative links
  • uv run --no-project python -m py_compile scripts/validate_notes.py
  • git diff --check

@DorianChn

Copy link
Copy Markdown
Owner Author

补充 commit 2bee75a:将年龄练习中的硬编码 2026 改为 date.today().year,避免示例随年份过期。已重新运行 scripts/validate_notes.py,2 个 Markdown 文件和全部相对链接通过。

@DorianChn

Copy link
Copy Markdown
Owner Author

独立验证补充(当前 main):

  • CI 等价检查(bundled Python)✅ validated 2 markdown files
  • 工作区无未提交改动 ✅
  • PR ci: validate learning note links #1scripts/validate_notes.py 差异已核对:会检查 Markdown 非空、相对链接存在且不逃逸仓库;年龄示例也改为动态年份。
  • GitHub 当前对该分支显示 no checks reported,以上为本地实际结果。

PR #1 与 main 无冲突,等待审核。

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