render-engine / render-engine/render-engine

check for required values

Aperta
#793 11 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

good first issue validation
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

  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.

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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.