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

Aberta Para iniciantes
  • Algo errado?
#267 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
88/100
Tipo de issue
Funcionalidade
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
github-actions, javascript, node.js
Domínio
ci-cd, testing

Direção de pesquisa

Leia .github/workflows/ci.yml e site/lib/lib.test.mjs e, em seguida, verifique como os jobs de CI existentes executam npm test. Adicione um job windows-latest que execute os testes em pull requests e cubra resolveRequest com os casos ..\\..\\x e %5c..%5c no win32. O trabalho estará concluído quando o job do Windows executar esses testes com sucesso em pull requests.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
JavaScript
Estrelas
58
Forks
15
Merge médio
1d 15h
PRs com merge (30d)
12

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de TMHSDigital/Free-Game-Dev-Assets

Todas as issues de TMHSDigital/Free-Game-Dev-Assets

Issues semelhantes

Mais issues de JavaScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.