Give needs-review entries a structured open question and surface it

Ouverte
#211 1 commentaire 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é
4/5
Temps estimé
3-5 jours
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
Plutôt claire
Activité
Active
Stack technique
javascript, node.js
Domaine
tooling

Piste de recherche

Start with the V9 - Deprecated: rule in site/checks.mjs and the age-only sorting in site/lib/reverify.mjs (lines 23-31), then see where needs-review is documented in CONTRIBUTING.md. Done means every needs-review entry carries a - Review: line that the checks enforce, the question appears on the entry page and in data.json, and the monthly re-verify issue lists them. The open decision on grandfathering existing entries needs a maintainer answer before implementing. Run npm run check to verify.

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

Description

enhancement priority: medium

Summary

43 entries are needs-review, none with a machine-readable statement of what is unresolved. deprecated requires a - Deprecated: line (V9); needs-review has no equivalent. The re-verify batch sorts by age only, so a recently checked needs-review entry waits months and contributors never see its question.

Related: #99

Where

  • CONTRIBUTING.md:16
  • site/lib/reverify.mjs:23-31
  • site/checks.mjs (V9 - Deprecated: rule)

Suggested fix

  • Add a - Review: marker line, enforced the way V9 is.
  • Show it on the entry page and in data.json.
  • Append an "Open questions" section to the monthly re-verify issue.

Open question: grandfather existing entries for one cycle while backfilling?

Acceptance criteria

  • Every needs-review entry states its open question
  • The re-verify issue lists them

Getting started

  • Relevant files: site/checks.mjs, site/lib/reverify.mjs, site/lib/entry-page.mjs
  • How to run tests: npm test (unit tests), npm run check (tests + validate + build). Node 22+, no dependencies.
  • Related docs: site/README.md, CONTRIBUTING.md

Filed by repo-audit/full-audit at 982fcd9. Severity: medium. Effort: M. audit-key: enhancement|site/checks.mjs|needs-review-marker

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

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.