Career models: the comment on ResumeItem.order is written twice #614

Closed
opened 2026-10-01 10:48:24 +00:00 by tiagoagueda · 1 comment
Owner

Found on 2026-09-29 in a read-only review of postulo at 95348908d, reading the career models.

Checked again on 2026-10-01 against main at 0a5cad587: still present.

The #: comment above ResumeItem.order is repeated: resume/models.py:36-38 and resume/models.py:39-41 are the same three lines, word for word. Anyone who edits the field's documentation has two copies to keep in step, and Sphinx-style #: doc comments render the text twice.

  • Where: resume/models.py:36-41 (ResumeItem.order).
  • What happens: the doc comment reads "The person's own order for the section, dense and exactly what the overview shows: … Hidden on the entry's form unless Settings > Appearance says otherwise." twice in a row.
  • Why it matters: a maintenance cost only; a later edit to one copy leaves the other stale.
  • Fix: delete lines 39-41.

Done when

  • resume/models.py carries the comment once; no test is needed.
Found on 2026-09-29 in a read-only review of postulo at 95348908d, reading the career models. Checked again on 2026-10-01 against `main` at `0a5cad587`: still present. The `#:` comment above `ResumeItem.order` is repeated: `resume/models.py:36-38` and `resume/models.py:39-41` are the same three lines, word for word. Anyone who edits the field's documentation has two copies to keep in step, and Sphinx-style `#:` doc comments render the text twice. - **Where:** `resume/models.py:36-41` (`ResumeItem.order`). - **What happens:** the doc comment reads "The person's own order for the section, dense and exactly what the overview shows: … Hidden on the entry's form unless Settings > Appearance says otherwise." twice in a row. - **Why it matters:** a maintenance cost only; a later edit to one copy leaves the other stale. - **Fix:** delete lines 39-41. ## Done when - `resume/models.py` carries the comment once; no test is needed.
tiagoagueda added this to the 0.6.0 milestone 2026-10-01 10:48:24 +00:00
Author
Owner

Landed on main with a green Forgejo CI run. Its entry in the CHANGELOG's Unreleased section ends in (#614).

Landed on main with a green Forgejo CI run. Its entry in the CHANGELOG's Unreleased section ends in (#614).
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Postulo/postulo#614
No description provided.