Flagsmith/flagsmith

Using django-lifecycle hooks results in `error: Untyped decorator makes function "x" untyped [misc]`

Open

#5 240 ouverte le 24 mars 2025

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)Python (3 475 stars) (264 forks)batch import
good first issue

Description

When using the @hook decorator from django-lifecycle on a model method, mypy throws the following error:

error: Untyped decorator makes function "x" untyped [misc]

This means that, currently, all methods decorated with @hook(...) are annotated with # type: ignore[misc]. The goal of this issue is to remove those type: ignore[misc] comments, while still having mypy pass its validation. Depending on the fix, it may be necessary to split the work up into multiple PRs as there are a lot of instances of this across the codebase.

Here is an example: https://github.com/Flagsmith/flagsmith/blob/main/api/features/models.py#L142

Guide contributeur

Stack technique
python
Domaine
backend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
PythonmypyDjangodecorators
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45
Direction de recherche
Examine the linked file and other occurrences of @hook decorator across the codebase. Investigate how django lifecycle defines the decorator and consider if a mypy plugin or type stubs can be added. Also check for any existing discussions in the comments on potential fixes.