Move optional dependencies from dev_requirements to pyproject.toml#239
Move optional dependencies from dev_requirements to pyproject.toml#239FreddyFox892 merged 19 commits intomainfrom
dev_requirements to pyproject.toml#239Conversation
|
@hf-kklein ich verstehe nicht warum der auf einmal bei pylint meckert, vorallem auch die ganzen pylint disable sachen nicht frisst |
hf-kklein
left a comment
hf-kklein
left a comment
There was a problem hiding this comment.
DEr untershceid scheint ja zu sein, dass die installation über
pip install .[foo]anders als die installation über
pip install -r dev_requirements/foo-requirements.txtteile des build prozesses durchläuft, wobei dieses version file angelegt wird. wie du das umgehst und warum das bei anderen packages kein problem ist 🤷♂️ das weiß ich leider nicht. vergleichende CI-Wissenschaften FTW?
| [tool.pylint."MASTER"] | ||
| ignore = ["src/ebdamame/version.py"] |
There was a problem hiding this comment.
was ist denn das? du hast die datei doch selbst angelegt in diesem PR? wenn sie nicht da wäre, müsstest du sie auch nicht ignorieren ;)
There was a problem hiding this comment.
edit: so leicht ist es nicht. #240 selbst wenn man die datei wieder entfernt, wird sie beim installarierne der optionalen dependencies trotzdem angelegt und führt dann hinten raus zu problemen.
There was a problem hiding this comment.
|
ähnliches problem haben wir auch hier |
| template = ''' | ||
| version = "{version}" | ||
| ''' |
There was a problem hiding this comment.
@hf-kklein ich glaube dass ding sorgt dafür dass dir die formatierung gerettet wird
There was a problem hiding this comment.
Erkenntnisse aus den Vergleichenden Ci-Wissenschaft, da war das selbe linter problem im catover tool aber ohne formatting problems^^
| black . --check | ||
| isort . --check | ||
|
|
||
| [testenv:spell_check] |
There was a problem hiding this comment.
wenn wirihn hier einführen, können wir ihn im nächsten PR auch verwenden in der ci?
2 participants
No description provided.