StackGuardian/tirith

Remove required_ from meta.required_provider and preserve backward compatibility

Aperta

#166 aperta il 8 ago 2024

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Python (41 fork)auto 404
good first issue

Metriche repository

Star
 (168 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We need to remove the required_ prefix from the meta.required_provider key in policies. The new key should be meta.provider. Also, we must ensure backward compatibility so that existing policies using meta.required_provider continue to function correctly.

  • Update the core, specifically the function start_policy_evaluation_from_dict to handle the above case
  • Write tests to ensure that policies with required_provider, and provider, both work.

Guida contributor

Direzione di ricerca
Trova la funzione `start policy evaluation from dict` nel codice principale e aggiornala per leggere sia la chiave `meta.required provider` che `meta.provider`, favorendo la nuova chiave. Scrivi test per verificare che entrambe le chiavi funzionino correttamente.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60