StackStorm/st2

Remove deprecated "timer" section in st2.conf

Ouverte

#4 425 ouverte le 5 nov. 2018

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)Python (740 forks)batch import
complexity:easyenhancementgood first issuerefactor

Métriques du dépôt

Stars
 (5 794 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Le problème nécessite de supprimer la section obsolète `timer` de `st2.conf` et de mettre à jour `config.py` pour ne plus y faire référence. La liste TODO est claire : supprimer de config.py, mettre à jour le code affecté et mettre à jour les notes de mise à niveau. Il n'y a pas de commentaires ni de PR liés, donc le contributeur doit examiner la base de code pour toutes les références à la section `timer` et assurer une suppression propre. Les notes de mise à niveau doivent refléter la suppression. Il s'agit d'un changement localisé sans dépendances.
Stack technique
python
Domaine
backend
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60