matrix-org / matrix-org/mscbot-python

mscbot may not start FCP when using `@mscbot reviewed`

Ouverte
#19 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug
Langage dominant
Python
Étoiles
7
Forks
2
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

One can use @mscbot reviewed to automatically check the box for themselves in an ongoing FCP status comment. The bot will also unconditionally ignore all comments and comment edits from itself.

When designing the reviewed command, it was incorrectly assumed that updating the status comment would then have the knock-on effect of starting FCP if enough ticks had happened. Instead, the edit is ignored as it comes from the bot.

A fix would simply involve manually calling _process_status_comment_update_with_body below https://github.com/matrix-org/mscbot-python/blob/fd6027708fd6ce08767556689470a6759c8af4ef/command_handler.py#L169-L172

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans command_handler.py autour des lignes 169–172 et suivez le traitement du commentaire d’état du FCP en cours pour la commande examinée. Assurez-vous que la mise à jour du commentaire d’état du bot est traitée de sorte que FCP démarre lorsque suffisamment de ticks se sont produits, puis vérifiez le comportement avec les tests pertinents du projet.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
tooling
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
50/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.