GoogleContainerTools/skaffold

remove deprecated `err` in all v1 protos.

Aperta

#5371 aperta il 8 feb 2021

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Go (1416 fork)batch import
area/api_v2area/deprecationbugbash/q4-2022help wantedkind/feature-requestkind/todopriority/p3

Metriche repository

Star
 (12.822 stelle)
Metriche merge PR
 (Merge medio 3g 6h) (16 PR mergiate in 30 g)

Descrizione

In Jun 2020, we introduced actionalErr proto and deprecated err. Its been 6 months and hence, remove the field from all proto defs. https://github.com/GoogleContainerTools/skaffold/blob/master/proto/skaffold.proto#L172

Guida contributor

Direzione di ricerca
Innanzitutto, individua il file proto/skaffold.proto citato nell'issue (riga 172) per vedere il campo deprecato `err` e la sua sostituzione `actionalErr`. Quindi, cerca tutte le occorrenze del campo `err` nei file .proto nella directory v1. Rimuovi la definizione del campo e i commenti correlati. Assicurati che nessun'altra parte della codebase (ad esempio, il codice Go generato) dipenda da esso, ma poiché è stato deprecato 6 mesi fa, dovrebbe essere sicuro. Rivedi i due commenti nell'issue per eventuali passaggi aggiuntivi o preoccupazioni sollevate dai manutentori. Infine, aggiorna eventuali codice protobuf generato, se necessario.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Protocol Buffers
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80