Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

repo: 295 of 416 catalog entries have CRLF line endings and there is no .gitattributes

Fermée
  • Quelque chose ne va pas ?
#140 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é
4/5
Temps estimé
3-5 jours
Accessibilité débutants
52/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
git, javascript
Domaine
tooling

Piste de recherche

Start by reading site/lib/frontmatter.mjs and markdown.mjs at the cited lines, then inspect validate.mjs, CONTRIBUTING, and .git-blame-ignore-revs. Add a line ending policy, renormalize tracked files, enforce it in validation, and document it. Done when git ls-files --eol shows no CRLF or mixed files and a CRLF fixture fails npm run validate with a clear message.

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

Description

dx priority: medium

Why it matters

The parsers tolerate \r\n (site/lib/frontmatter.mjs:92, markdown.mjs:104), so nothing breaks today, but the repo has no line-ending policy: git ls-files --eol reports 295 files as i/mixed (CRLF body, LF final line) and 5 as i/crlf, next to 225 i/lf. Contributors on Linux and macOS get whole-file diffs when their editor normalises, npm run new-entry output and hand-edited files disagree, and a future helper that splits on \n alone will silently keep a \r in a field value. It is the kind of friction a first-time contributor hits and gives up on.

Evidence (at 69387df)

$ git ls-files --eol | awk '{print $1}' | sort | uniq -c
    5 i/crlf
  225 i/lf
  295 i/mixed

All 295 mixed files are under catalog/ (tools 65, 3d 60, 2d 49, audio 25, characters 24, fonts 23, shaders-vfx 15, video 13, environment 11, animation 10). No .gitattributes exists; core.autocrlf is unset.

Suggested approach

Add .gitattributes with * text=auto eol=lf plus binary for *.webp, *.png, *.woff2; run git add --renormalize . in one commit titled "Normalise line endings" and list it in .git-blame-ignore-revs. Add a one-line check in validate.mjs that rejects a \r in any catalog or docs file, so the policy is enforced rather than assumed.

Done when

git ls-files --eol shows no i/crlf or i/mixed; a CRLF fixture fails npm run validate with a clear message; CONTRIBUTING mentions the policy in one sentence.

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

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.