Skip to content

Fix docs-consistency test assumptions for current README set - #2

Closed
0liveiraaa with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job
Closed

Fix docs-consistency test assumptions for current README set#2
0liveiraaa with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Jul 24, 2026

Copy link
Copy Markdown

The build-and-test (20.x) Actions job failed because docs-consistency tests referenced README-EN.md (not present in repo) and expected a Chinese debug-logging phrase in README.md (now English). This PR aligns test expectations with the current documentation layout/content.

  • Root-cause alignment in docs test inputs

    • Switched documentation file lists from README-EN.md to README-zh.md in tests/docs-consistency.test.ts.
    • This removes ENOENT failures during markdown consistency checks.
  • Assertion update for README language

    • Updated the debug-logging default assertion to match the current README.md wording:
      • Debug log files are not written by default.
    • Preserved the debug: true presence check.
const publicDocs = [resolve(root, "README.md"), resolve(root, "README-zh.md"), ...markdownFiles(docsRoot)];

expect(readme).toContain("Debug log files are not written by default.");

Copilot AI changed the title [WIP] Fix the failing GitHub Actions job build-and-test (20.x) Fix docs-consistency test assumptions for current README set Jul 24, 2026
Copilot AI requested a review from 0liveiraaa July 24, 2026 18:28
@0liveiraaa
0liveiraaa marked this pull request as ready for review July 24, 2026 18:28
@0liveiraaa 0liveiraaa closed this Jul 24, 2026
Copilot stopped work on behalf of 0liveiraaa due to an error July 24, 2026 18:33
Copilot AI requested a review from 0liveiraaa July 24, 2026 18:33
@0liveiraaa
0liveiraaa deleted the copilot/fix-failing-github-actions-job branch July 24, 2026 18:33
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.

2 participants