Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

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

Chiusa
  • Qualcosa non va?
#140 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
52/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
git, javascript
Ambito
tooling

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
JavaScript
Stelle
58
Fork
15
Merge medio
18h 42m
PR unite (30g)
11

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di TMHSDigital/Free-Game-Dev-Assets

Tutte le issue di TMHSDigital/Free-Game-Dev-Assets

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.