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

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#267 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
88/100
Type d'issue
Fonctionnalité
Clarté
Clairement spécifiée
Activité
Active
Stack technique
github-actions, javascript, node.js
Domaine
ci-cd, testing

Piste de recherche

Lisez .github/workflows/ci.yml et site/lib/lib.test.mjs, puis examinez comment les tâches CI existantes exécutent npm test. Ajoutez une tâche windows-latest qui exécute les tests sur les pull requests et couvrez resolveRequest avec les cas ..\\..\\x et %5c..%5c sur win32. Le travail est terminé lorsque la tâche Windows exécute ces tests avec succès sur les pull requests.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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

Langage dominant
JavaScript
Étoiles
58
Forks
15
Merge moyen
1 j 15 h
PR mergées (30 j)
12

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de TMHSDigital/Free-Game-Dev-Assets

Toutes les issues de TMHSDigital/Free-Game-Dev-Assets

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.