Skip to content

fix: resolve CI failures#2

Merged
jianliang00 merged 1 commit into
mainfrom
p/wangjianliang/external-autolink
May 25, 2026
Merged

fix: resolve CI failures#2
jianliang00 merged 1 commit into
mainfrom
p/wangjianliang/external-autolink

Conversation

@jianliang00

Copy link
Copy Markdown
Owner

Summary

  • Rename template marker files to Windows-compatible names.
  • Keep support for the previous marker names when reading external templates.
  • Update Mocha so the test runner works with the current Node latest version.

Root cause

Windows runners could not check out template files whose names contained angle brackets. The Node latest job also resolved to Node 26, where Mocha 10's yargs dependency failed during ESM loading.

Validation

  • pnpm run build
  • pnpm run test
  • npx -y node@26.2.0 ./node_modules/mocha/bin/mocha.js --forbid-only "test/**/*.test.ts"
  • git diff --check

@jianliang00 jianliang00 force-pushed the p/wangjianliang/external-autolink branch from b31405d to 80a72c4 Compare May 25, 2026 13:15
@jianliang00 jianliang00 force-pushed the p/wangjianliang/external-autolink branch from 80a72c4 to f3dc6b6 Compare May 25, 2026 13:18
@jianliang00 jianliang00 marked this pull request as ready for review May 25, 2026 13:25
@jianliang00 jianliang00 merged commit 059a451 into main May 25, 2026
6 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