CI never runs on Windows although path and case handling is platform-specific

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#267 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
88/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, javascript, node.js
Lĩnh vực
ci-cd, testing

Hướng nghiên cứu

Đọc .github/workflows/ci.yml và site/lib/lib.test.mjs, sau đó kiểm tra cách các job CI hiện có chạy npm test. Thêm một job windows-latest chạy kiểm thử trên các pull request và kiểm thử resolveRequest với các trường hợp ..\\..\\x và %5c..%5c trên win32. Hoàn thành khi job Windows chạy thành công các kiểm thử đó trên pull request.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

good first issue priority: low tests

Summary

Both CI jobs use ubuntu-latest, but the code has about 28 places handling path.sep, backslashes or realpathSync.native, plus case-insensitive filesystem logic, and contributors work on Windows.

Where

  • .github/workflows/ci.yml:17, :56
  • examples: site/build.mjs:290, site/validate.mjs:126, site/serve.mjs:48, :54, site/lib/links.mjs (caseMismatch)

Impact

A separator or case regression in containment checks such as resolveRequest passes CI and only breaks local Windows runs.

Suggested fix

Add a windows-latest job running npm test (about 8 s), or the full npm run check with fetch-depth: 0.

Acceptance criteria

  • windows-latest job runs the tests on PRs
  • resolveRequest cases for ..\\..\\x and %5c..%5c on win32

Getting started

  • Relevant files: .github/workflows/ci.yml, site/lib/lib.test.mjs
  • Run the tests: npm test; the full local gate is npm run check (Node 22+, no install step)
  • Related docs: CONTRIBUTING.md, site/README.md

Related: #139

Filed by repo-audit/full-audit at 4adb944. Severity: low. Effort: S. audit-key: testing|.github/workflows/ci.yml|windows

Ngôn ngữ chính
JavaScript
Star
58
Fork
15
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
12

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của TMHSDigital/Free-Game-Dev-Assets

Tất cả issue của TMHSDigital/Free-Game-Dev-Assets

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.