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

Events/freies hacken #74

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Events/freies hacken #74

wants to merge 9 commits into from

Conversation

timherrm
Copy link
Contributor

@timherrm timherrm commented Nov 6, 2023

No description provided.

@MG-5
Copy link
Member

MG-5 commented Nov 6, 2023

Wie in Issue schon geschrieben: Für mich ist das keine gute Idee, jeden Mittwoch mit Freies Hacken zu belegen, weil mein Kalender dann unübersichtlich wird und ich dann wahrscheinlich deabbonnieren würde.

@timherrm timherrm marked this pull request as draft November 6, 2023 11:21
@penguineer
Copy link
Member

penguineer commented Nov 6, 2023

Über Tags könnte man das sortieren.

Wie im Issue auch geschrieben, bevor wir viele events generieren, die wir alle nochmal anfassen müssen, sollten wir Startzeit/Dauer klären.

Siehe #78

@penguineer
Copy link
Member

Offtopic: Damit wir die Merges nicht wieder bekommen, sollte beim Update die Option "Update with rebase" gewählt werden. Das lässt sich leider nicht voreinstellen.

@timherrm
Copy link
Contributor Author

timherrm commented Nov 9, 2023

Ich habe jetzt die Events gelöscht und würde erstmal nur das Script in den Tools Ordner mergen. Wenn wir dann entschieden haben wie es mit den Events mit der Zeit weitergeht, dann mache ich nen neuen PR auf.

@24367dfa
Copy link
Member

24367dfa commented Nov 9, 2023

Nimm Mal das draft raus. Und mach Mal eine git rebase -i main und wirf die commits raus, die du nicht mit mergen willst.

@timherrm timherrm marked this pull request as ready for review November 9, 2023 10:52
@timherrm
Copy link
Contributor Author

timherrm commented Nov 9, 2023

Nimm Mal das draft raus. Und mach Mal eine git rebase -i main und wirf die commits raus, die du nicht mit mergen willst.

so richtig?

@timherrm timherrm marked this pull request as draft November 9, 2023 11:49
@timherrm
Copy link
Contributor Author

timherrm commented Nov 9, 2023

Dann ist das doch mal noch WIP. Der Teil, dass die Events in Jahresordnern landen fehlt auch noch.

@timherrm timherrm self-assigned this Nov 9, 2023
@timherrm
Copy link
Contributor Author

timherrm commented Nov 9, 2023

@24367dfa Kannst du so nochmal drüber schauen?

@timherrm timherrm marked this pull request as ready for review November 9, 2023 12:26
Copy link
Member

@penguineer penguineer left a comment

Choose a reason for hiding this comment

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

Das klingt vllt nach Nitpicking, aber ich habe mir inzwischen angewöhnt, für Python-Scripte immer auch einen Docker-Container zu definieren und im README die entsprechenden Aufrufe (mit Volume Mount) zu definieren. Wir tun uns damit mittel- und langfristig einen großen Gefallen.

Hintergrund ist, dass es ziemlich nervig sein kann, die passenden Dependencies im System zu haben, insbesondere weil es Bibliotheken gibt, die miteinander im Konflikt stehen.

Da auch virtual environments gerade nicht zuverlässig deploybar sind, ist ein Container die zuverlässigste Lösung.

Heißt konkret:

  • Tool-Spezifisches Verzeichnis, damit wir nicht den Überblick verlieren (git mv)
  • Dockerfile
  • Entsprechender Container-Aufruf in der README

Wenn Du dabei Unterstützung brauchst, sag bitte Bescheid.

```
Output:

```bash
Copy link
Member

Choose a reason for hiding this comment

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

Wollen wir diesen Output wirklich im Repo haben?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mir ist das egal, ich kann es auch einfach wieder löschen. Möchte nur verstehen, was dein Gedanke dahinter ist, das zu löschen, damit ich es fürs nächste Mal weiß 😄

Copy link
Member

Choose a reason for hiding this comment

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

Ich hab mich gefragt, was wir mit der Info im README machen - geht es darum, noch zu wissen, was dafür passiert ist? Dann würde ich die Info eher in die Commit-Message oder den PR schreiben.

Copy link
Contributor Author

@timherrm timherrm Nov 10, 2023

Choose a reason for hiding this comment

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

Für mich war es eher ein "so führst du es aus" und "das sollte dann dabei rauskommen, wenn alles funktioniert hat".
Hat für mich als Unwissender halt Sinn gemacht das da hin zu schreiben :D
Aber ja man sieht ja im Zweifel ob eine Fehlermeldung kommt oder Dateien in den Ordner geschrieben werden

@penguineer
Copy link
Member

@timherrm Nach dem Git-Workshop: Hast Du Lust, die Commits ein wenig zusammenzufassen? ;)

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.

4 participants