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

Lint Django templates with djLint and enable it as a prek hook

Aperta
  • Qualcosa non va?
#252 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 12 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
68/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
django, python
Ambito
frontend, testing-qa, tooling

Direzione di ricerca

Start with the 13 templates listed in the issue and run uvx --from djlint==1.46.2 djlint pythonie --lint --profile=django to reproduce the reported findings. Then review pyproject.toml, .pre-commit-config.yaml, CONTRIBUTING.md, and CLAUDE.md for the shared configuration and hook documentation. Done means the lint and prek checks pass, the meetup sponsors and bundled theme script still work, and the listed test command succeeds.

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

Descrizione

enhancement

Context

The repository runs its git hooks with prek (see .pre-commit-config.yaml, added in #249). The hooks cover Python (ruff, django-upgrade, Django checks, missing migrations) and generic files, but nothing checks the Django templates.

djLint is a linter and formatter for HTML templates with template syntax (Django profile). It was evaluated in #249 and left out because it does not pass on the current templates. This issue tracks fixing the templates and then enabling djLint as a prek hook.

Scope

13 templates:

pythonie/core/templates/404.html
pythonie/core/templates/500.html
pythonie/core/templates/base.html
pythonie/core/templates/footer.html
pythonie/core/templates/header.html
pythonie/core/templates/navbar.html
pythonie/core/templates/navbar_tree.html
pythonie/core/templates/core/home_page.html
pythonie/core/templates/core/meetup.html
pythonie/core/templates/core/segment.html
pythonie/core/templates/core/simple_page.html
pythonie/core/templates/core/sponsor.html
pythonie/templates/wagtailembeds/embed_frontend.html

Current state (djLint 1.46.2, master at 90cf69d)

uvx --from djlint==1.46.2 djlint pythonie --lint --profile=django reports 12 errors:

File Line Rule Message
core/templates/500.html 5 H005 <html> should have a non-empty lang attribute
core/templates/base.html 8 H005 <html> should have a non-empty lang attribute
core/templates/base.html 9, 45, 63 H014 Extra blank lines
core/templates/base.html 43, 44, 78 H024 Omit type on scripts and styles
core/templates/core/meetup.html 5 H025 Tag seems to be an orphan (<div class="col-md-12">)
core/templates/header.html 8 H013 <img> should have an alt attribute
core/templates/navbar.html 21, 22 T001 Tags should be wrapped in whitespace ({%include ...%}, {%endfor%})

djlint pythonie --check --profile=django reports that all 13 files would be reformatted (about 216 changed lines in total).

Real markup bugs in core/templates/core/meetup.html

The H025 warning points at actual bugs in the sponsor block of the meetup list:

  1. Line 24: <div class="col-md-8">has a stray backtick, so the browser parses a brokenclassattribute and thecol-md-8` column class is not applied.
  2. <div class="name"> (line 25) is never closed: 5 <div> are opened inside the loop body (col-md-12, sponsor-list, sponsor, col-md-8, name) and only 4 are closed, so the browser closes the elements implicitly and the layout of the sponsor list depends on error recovery.

Tasks

Split the work into separate commits so the review stays easy:

1. Fix the lint errors (fix(templates): ...)
  • meetup.html: remove the stray backtick and close <div class="name"> right after the sponsor name block (before the {% if sponsor.note %} caption, or after it if the caption belongs to the name block; check the rendering).
  • base.html and 500.html: add lang="en" to <html>. In 500.html the <html> tag sits inside IE conditional comments (<!--[if ...]>); the non-IE branch (line 5) is the one djLint flags, add lang="en" to every branch for consistency (or drop the IE conditional comments, IE is no longer supported by Bootstrap/the site).
  • header.html: add alt="Python Ireland" to the logo <img>.
  • base.html: drop type="text/javascript" on the three <script> tags (lines 43, 44, 78). Line 78 is inside {% compress js %}: django-compressor handles scripts without a type attribute as JavaScript, but verify it by rendering a page with COMPRESS_ENABLED=True (or running manage.py compress if offline compression is used) and checking that theme-toggle.js is still bundled.
  • navbar.html: {%include "navbar_tree.html" %} becomes {% include "navbar_tree.html" %}, {%endfor%} becomes {% endfor %}.
  • base.html: remove the extra blank lines.

After this commit, djlint pythonie --lint --profile=django must report 0 errors.

2. Reformat the templates (style(templates): ...), optional

Run djlint pythonie --reformat --profile=django and commit the result on its own. It changes whitespace and indentation only, but djLint also reflows some inline content (for example pythonie/templates/wagtailembeds/embed_frontend.html is collapsed to a single line), so check the rendered pages visually (home page, a simple page, the navbar with nested pages, the meetup list with sponsors, the 404 page).

If the formatter output is not acceptable on some templates, keep only the linter (task 3 without djlint-reformat-django) and mention it in the PR.

3. Configure djLint and enable the prek hook (chore(dev): ...)

  • Add the configuration to pyproject.toml so the CLI, the hook and editors share it:

    [tool.djlint]
profile = "django"
# ignore = "H030,H031"  # only if some rules are deliberately skipped, with a comment explaining why

  • Add the hooks to .pre-commit-config.yaml, pinned on a tag (latest at the time of writing: v1.46.2, check with gh api repos/djlint/djLint/tags --jq '.[0].name'):

    - repo: https://github.com/djlint/djLint
  rev: v1.46.2
  hooks:
    - id: djlint-reformat-django  # only if task 2 is done
    - id: djlint-django

    djLint only targets .html files by default. The vendored assets under pythonie/core/static/{css,fonts,js} are already excluded by the global exclude of the config.

  • Nothing to change in CI: the prek job in .github/workflows/test.yml runs every hook with --all-files. Renovate already updates the hook revisions (the pre-commit manager is enabled in renovate.json, grouped in the "pre-commit hooks" PR).

  • Document djLint in the list of hooks in CONTRIBUTING.md (section "Git Hooks (prek)") and in CLAUDE.md (section "Git Hooks (prek)").

Acceptance criteria

  • prek run --all-files passes, djLint hooks included
  • djlint pythonie --lint --profile=django reports 0 errors
  • The meetup list with sponsors renders correctly (the two bugs in meetup.html are fixed)
  • theme-toggle.js is still loaded and bundled by django-compressor
  • uv run python pythonie/manage.py test pythonie --settings=pythonie.settings.tests passes (run it with env -u DATABASE_URL if a DATABASE_URL is exported in the shell, otherwise the test settings use that database)
  • The CI prek and test jobs are green
  • Separate commits for the fixes, the optional mass reformat and the hook configuration

Out of scope

  • Redesigning the templates or upgrading Bootstrap.
  • base.html loads Typekit from //use.typekit.net/qgj1xay.js. Whether this kit still exists is worth checking, but in a separate issue.
Lingua principale
Python
Stelle
17
Fork
27
Merge medio
59m
PR unite (30g)
13

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 PythonIreland/website

Tutte le issue di PythonIreland/website

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.