Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin CI Dependencies #474

Closed
hf-kklein opened this issue Jul 3, 2023 · 0 comments · Fixed by #536
Closed

Pin CI Dependencies #474

hf-kklein opened this issue Jul 3, 2023 · 0 comments · Fixed by #536
Assignees
Labels
bug Something isn't working Python Specific 🐍 a problem that is specific to this python implementation and not the bo4e data model

Comments

@hf-kklein
Copy link
Contributor

hf-kklein commented Jul 3, 2023

Es ist etwas nervig, dass bspw. linter versionen nicht gepinned sind und (mittlerweile regelmäßig) die CI failed, weil eine neue linter version veröffentlicht, der code aber nicht entsprechend angepasst wurde.

Vorlage für das Pinnen wäre https://github.com/Hochfrequenz/python_template_repository
(konkret der Ordner dev_requirements und die die Verweise der tox.ini darauf).

D.h. anstatt z.b. pylint in der linting umgebung immer in der (implizit) aktuellsten version zu installieren sollten wir auch die requirements der CI sauber per jeweils einem requirements.in und einer requirements.txt verwalten.

Wir wollen damit verhindern, dass linter und type-check probleme unerwartet auftreten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Python Specific 🐍 a problem that is specific to this python implementation and not the bo4e data model
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants