alteryx / alteryx/evalml

Decrease runtime of lint job

Aperta
#2,779 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement performance spike testing
Lingua principale
Python
Stelle
850
Fork
96
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

#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).

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.