StackStorm/st2

Remove deprecated "timer" section in st2.conf

Aperta

#4425 aperta il 5 nov 2018

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)Python (740 fork)batch import
complexity:easyenhancementgood first issuerefactor

Metriche repository

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

Descrizione

In v2.9.0 we deprecated timer section in st2.conf in favor of new timersengine section.

We need to remove old and deprecated timer section in st2.conf.

The current approach is also confusing - timer section has been deprecated and timerengine is the preferred approach yet the how code is structured, "timer" section actually has precedence because of the way how our code is structured.

TODO

  • Remove from config.py
  • Update affected code
  • Update upgrade notes

Guida contributor

Direzione di ricerca
Il problema richiede la rimozione della sezione deprecata `timer` da `st2.conf` e l'aggiornamento di `config.py` per non farvi più riferimento. L'elenco TODO è chiaro: rimuovere da config.py, aggiornare il codice interessato e aggiornare le note di aggiornamento. Non ci sono commenti o PR collegati, quindi il contributore dovrebbe esaminare la codebase per tutti i riferimenti alla sezione `timer` e assicurarsi una rimozione pulita. Le note di aggiornamento devono riflettere la rimozione. Si tratta di una modifica localizzata senza dipendenze.
Tech stack
python
Dominio
backend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60