chroma-core/chroma

Enable mypy lint in CI

Offen

#2.054 geöffnet am 24.04.2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (2.247 Forks)batch import
by-chromacleanupgood first issue

Repository-Metriken

Stars
 (27.958 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 16h) (80 gemergte PRs in 30 T)

Beschreibung

I recently enabled most of our lints in CI: https://github.com/chroma-core/chroma/pull/2046.

I didn't enable mypy in that PR because it looked like it required a few manual changes to pass the lint. We need to enable this lint by updating .github/workflows/chroma-lint.yml and making the appropriate changes to pass the lint (test locally with pre-commit run --all-files mypy).

Contributor Guide

Research-Richtung
Aktualisieren Sie .github/workflows/chroma lint.yml, um mypy hinzuzufügen, und führen Sie dann lokal pre commit run all files mypy aus, um Lint Fehler zu beheben.
Tech Stack
python
Domain
ci cd
Issue Type
Wartung
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGitCI/CD
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70