Skip to content

🎉An initial Draft #1

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

Merged
merged 33 commits into from
Mar 26, 2021
Merged

🎉An initial Draft #1

merged 33 commits into from
Mar 26, 2021

Conversation

hf-kklein
Copy link
Contributor

@hf-kklein hf-kklein commented Mar 8, 2021

Ich dachte, wir könnten mittlerweile ein Template-Repository gebrauchen also eine Best-Practise-Vorlage für neue Python-Projekte. Man kann dann dieses Ding als Template wählen:
grafik

und hätte out-of-the-box direkt eine tox-datei, ein Test-Setup und vier Github Actions.

@hf-kklein hf-kklein requested a review from a team March 8, 2021 11:34
@hf-kklein hf-kklein self-assigned this Mar 8, 2021
@hf-kklein hf-kklein requested a review from hf-krechan March 10, 2021 08:30
Copy link
Contributor

@hf-aschloegl hf-aschloegl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Echt eine gute Idee! Dann gibt es immer eine weiterentwickelte und aktuelle Grundlage für neue Projekte.
Hab ein paar kleinere Sachen und ein paar Grundsatzfragen angemerkt.

hf-krechan and others added 10 commits March 10, 2021 11:17
the actual pre-commit settings do:
* check yaml format
* fix end-of-file 
* remove trailing whitespace
* check for black formatting style
Co-authored-by: hf-aschloegl <[email protected]>
Co-authored-by: hf-aschloegl <[email protected]>
Co-authored-by: hf-aschloegl <[email protected]>
Co-authored-by: hf-aschloegl <[email protected]>
Co-authored-by: hf-aschloegl <[email protected]>
Co-authored-by: hf-aschloegl <[email protected]>
Copy link
Contributor

@hf-krechan hf-krechan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gefällt mir schon richtig gut.
Habe noch pre-commit hinzugefügt.
Vielleicht ist es noch sinnvoll auch isort reinzupacken, da VS Code die imports nicht so schön sortiert wie PyCharme (was vermutlich auch nur isort verwendet)

@hf-krechan
Copy link
Contributor

Allgemein würde ich auch gerne versuchen so viele Settings wie möglich in die pyproject.toml Datei einzubringen.
Denke im Laufe der Zeit werden alle großen Tools auch dort ihre Settings einarbeiten.

Copy link
Contributor

@hf-aschloegl hf-aschloegl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Schaut gut aus, nur noch eine kleine Nachfrage zu dem tox-Befehl

@hf-kklein
Copy link
Contributor Author

Also wollen wir, wenn wir den tox Befehl ausführen, auch dev nochmal laufen lassen?

Ich lasse das lokal eh kaum laufen 😀 Ich nehme was immer ihr wollt und auf Github läuft

@hf-krechan
Copy link
Contributor

Ja dev sollte nicht in der env-Liste enthalten sein. Wenn man nur tox als Befehl ausführt, sollten nur die Environments ausgeführt werden, die auch wirklich etwas tun, also testing, coverage und linting.

@hf-krechan hf-krechan removed their request for review March 26, 2021 09:37
@hf-kklein hf-kklein merged commit 5c0cd86 into main Mar 26, 2021
@hf-kklein hf-kklein deleted the initial_draft branch March 26, 2021 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants