Flagsmith / Flagsmith/flagsmith
ProjectIntegration types fail typecheck
- Vorherrschende Sprache
- Python
- Sterne
- 6.6k
- Forks
- 567
- Ø Merge
- 1 T. 14 Std.
- Gemergte PRs (30 T.)
- 124
Beschreibung
> note: I guess there's nothing to be done here except making ProjectIntegrationBaseViewSet generic (which is I appreciate is way out of scope for this PR).
_Originally posted by @khvn26 in https://github.com/Flagsmith/flagsmith/pull/7239#discussion_r3084947039_
A/C TBD, but overall goal is reduce `type: ignore` comments in integrations code — thus making it bug-safer.
Beitragsleitfaden
Rechercherichtung
Beginne damit, ProjectIntegration und ProjectIntegrationBaseViewSet zu finden, und untersuche dann den Integrationscode auf die im Issue erwähnten type: ignore-Kommentare. Die Akzeptanzkriterien sind noch TBD; kläre, welche Ignores und Typecheck-Fehler in den Umfang fallen, und überprüfe anschließend, dass die betroffenen Typen den Typecheck fehlerfrei bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100