python / python/mypy

perhaps when mypy should be re-run after --install-types, the exit code should be 3

Offen
#19,808 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

topic-configuration topic-error-reporting
Vorherrschende Sprache
Python
Sterne
20.6k
Forks
3.3k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Frankly, this is a mess. Code 2 should not be used for --install-types or blocking errors IMO - 2 is de-facto standard for "please do not invoke me like this". "Run again" and "invalid syntax" have so little in common that I struggle to understand the reason to combine them under one code.

Originally posted by @sterliakov in #19548

It would probably be acceptable to pick whatever we want to be the exit code --install-types gives when mypy needs to be re-run afterwards, but 3 seems a good choice.

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

Beginne damit, die mypy-Codebasis nach --install-types und der aktuellen Behandlung von Exit-Codes zu durchsuchen, und lies anschließend die Diskussion in Issue #19548 zur vorgesehenen Unterscheidung. Erledigt ist die Aufgabe, wenn mypy den Exit-Code 3 zurückgibt, wenn --install-types Typen installiert und ein erneuter Lauf erforderlich ist, und das relevante Verhalten durch Tests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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