posit-dev / posit-dev/rsconnect-python

fix pyright errors

Offen
#774 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
37
Forks
28
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
7

Beschreibung

pyright was added to make lint in #567, but it was immediately suppressed with a leading - in the Makefile so it doesn't fail the build. Currently there are 110 pyright errors. I believe these are all things that we at one point intended to enforce but are not currently doing so.

This issue is to do cleanup of those 110 errors, and then turn on real enforcement by removing the -

https://github.com/posit-dev/rsconnect-python/blob/19f54159428daf20eb07b9c77b29823166538de1/Makefile#L59-L64

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit dem Makefile-Target in den Zeilen 59-64 und führen Sie den vorhandenen lint-Befehl aus, um die 110 pyright-Fehler zu erfassen. Arbeiten Sie die gemeldeten Python-Dateien durch und entfernen Sie anschließend das führende - aus dem pyright-Befehl. Als erledigt gilt die Aufgabe, wenn die Bereinigung abgeschlossen ist und make lint pyright erfolgreich erzwingt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.