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

create asserts a default page width that update leaves alone

Aperta
#184 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Attiva
Stack tecnologico
go
Ambito
cli

Direzione di ricerca

Start with resolveWidth in cmd/create/create.go and compare its default handling with the update path described in the issue. Read the principles documentation and the export and configuration guidance mentioned here. Done means one width policy is chosen, create and update agree or the exception is explicitly documented, and the relevant behavior is reflected in the project’s guidance.

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

Descrizione

create gives every new page a width of max when the file declares none. resolveWidth (cmd/create/create.go) walks --page-width, then frontmatter page_width, then the project file's page_width, and falls back to pagewidth.DefaultWidth when all three are silent. update does the opposite: an absent page_width makes no width request at all and leaves the live width alone.

That makes the two writing verbs disagree about what an absent field means, and it is the one thing keeping L9 (declared-metadata-is-asserted: a field the file declares is made true of the page; a field it omits leaves the page alone) from holding in both verbs. labels and page_status already hold in both.

The question is a decision rather than a bug fix:

  • Leave Confluence's own default. create makes no width request when nothing declares one, matching update and labels. A new page gets whatever Confluence gives a page created through the API.
  • Keep max as house style, but make it a declaration. For example, export and the docs recommend page_width: max in markfluence.yaml, and create asserts only what is declared.
  • Keep the current behavior and say in the principles doc that create asserts a default width, as an accepted exception.

Whichever it is, update and create should agree, or the difference should be written down on purpose.

Lingua principale
Go
Stelle
2
Fork
0
Merge medio
2h 50m
PR unite (30g)
52

Guida per i contributori

Apri la guida per i contributori

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 mozilla/markfluence

Tutte le issue di mozilla/markfluence

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.