alteryx / alteryx/evalml

Decrease runtime of lint job

Offen
#2,779 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement performance spike testing
Vorherrschende Sprache
Python
Sterne
850
Forks
96
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

#2670 introduced `pydocstyle` and `darglint` packages. The `darglint` package specifically increases the runtime of our lint job by a few minutes. While we were okay with this addition, I suspect that there's a way to cut down on this time. Some suggestions are:

- Find a way to only lint the changed files. @freddyaboulton mentioned that scikit-learn has a way to do so via bash script that might be worth exploring: https://github.com/alteryx/evalml/pull/2670#issuecomment-918578759
- Explore github pre-commit hooks, and only call the lint job on files modified (that are not test files, and .py files).

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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