render-engine / render-engine/render-engine
check for required values
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 61
- Fork
- 41
- Merge medio
- 4h 29m
- PR unite (30g)
- 10
Descrizione
Happy to move this over to a discussion if necessary until we have a solid plan but based on #236, each object has required items that need to be defined on initialization.
This varies based on the object (not the object type).
Perhaps it would be good to create a validation step in there that checks for all of the required attributes.
To Implement
BaseObject would need a method validates that would be used to raise an AttributeError (?).
Since Page and Collection objects inherit from BaseObject then custom parsers can override it.
We can also apply the following validations.
Page - No validation
BlogPost - requires both date and content_path attributes.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia leggendo le definizioni di BaseObject, Page, Collection e BlogPost e consulta la issue #236 per esaminare il comportamento esistente degli elementi obbligatori. Determina un contratto di validazione e la scelta dell’eccezione prima di implementarlo, inclusi i requisiti di BlogPost per date e content_path e l’assenza di validazione per Page. Il lavoro è completato quando l’inizializzazione rifiuta in modo coerente gli attributi obbligatori mancanti e il comportamento è coperto da test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100