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

docs(Buehler): Reflexion Sprint 2 #50

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

docs(Buehler): Reflexion Sprint 2 #50

wants to merge 1 commit into from

Conversation

robinb19
Copy link
Contributor

@robinb19 robinb19 commented Dec 16, 2024

See #49

@robinb19 robinb19 requested a review from codingluke December 16, 2024 08:23
@robinb19 robinb19 self-assigned this Dec 16, 2024
Copy link
Contributor

@codingluke codingluke left a comment

Choose a reason for hiding this comment

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

Lieber Herr Bühler

Es ist sichtbar, dass Sie die Kritik der letzten Reflexion nicht einfach kalt gelassen hat. So wurden die Fragen detaillierter beantwortet. Hin und da fehlt jedoch immer noch das wieso.

Die Aussage, dass das Deployment reibungslos war, zeigt auf, dass Sie sich nicht mit Herrn Raetzo kurzgeschlossen haben, welcher diese Aufgabe umgesetzt hat. Ebenso haben sie meinen PR dazu nicht gelesen. Hätten Sie dies gemacht, würden Sie nicht zu dieser Konklusion kommen.

Beste Grüsse und frohe Festtage
Lukas Hodel


Grundsätzlich waren die Themen gut gewählt, jedoch mit einigen Einschränkungen. **Docker** wurde genau zum richtigen Zeitpunkt behandelt, da es uns ermöglichte, unsere Applikation einfacher in AWS zu deployen. Der CI/CD-Schwerpunkt war ebenfalls passend, da wir in dieser Projektphase bereit waren, Automatisierungsschritte für Build und Deployment einzuführen.

Jedoch hätte das Thema **Formatierung und Linting** bereits früher behandelt werden können. Zu diesem Zeitpunkt hatten wir bereits viele Dateien geschrieben, die nachträglich angepasst werden mussten. Eine frühere Einführung hätte uns geholfen, konsistenter zu arbeiten. Man hätte eine erste Einführung zu Docker integrieren und Formatierung dort nur grob anschneiden können, um es später genauer zu behandeln.
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich werde es mir hinter die Ohren schreiben. Linting war zuerst am Anfang, da hatte ich das Gefühl es ist viel Theorie und schob es ans Ende, wenn schon was da ist zum Linten. Fast alle haben dies bemängelt. Wahrscheinlich schiebe ich es wieder nach vorne.

In diesem Sprint wurden mehrere wichtige Meilensteine erreicht:
1. **Deployment in AWS**: Unsere Applikation wurde erfolgreich in der AWS-Umgebung deployt.
2. **Docker-Optimierung**: Wir haben produktive Dockerfiles erstellt und verfeinert.
3. **Erste Tests**: Kleine Tests wurden geschrieben, um erste Fehler frühzeitig zu erkennen.
Copy link
Contributor

Choose a reason for hiding this comment

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

Was habt ihr bereits getestet?

### Was wurde in diesem Sprint nicht erreicht?

Trotz der Erfolge gab es auch Bereiche, in denen wir nicht wie geplant vorangekommen sind:
1. **Fertigstellung der Applikation**: Die Applikation wurde nicht vollständig fertiggestellt.
Copy link
Contributor

Choose a reason for hiding this comment

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

Was zum Beispiel?

Wichtig war, Probleme direkt anzusprechen und gemeinsam nach Lösungen zu suchen. Fehler wurden nicht kritisiert, sondern im Team erklärt, damit jeder daraus lernen konnte. Die offene und respektvolle Kommunikation hat dazu beigetragen, dass wir effizient arbeiten konnten.

**Was ist uns besonders gut gelungen, worauf sind alle richtig stolz?**
Besonders stolz sind wir auf das erfolgreiche Deployment in AWS. In anderen Teams gab es viele Fehler und Herausforderungen beim Deployment, während wir diesen Schritt vergleichsweise reibungslos geschafft haben.
Copy link
Contributor

Choose a reason for hiding this comment

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

haha, der war gut. Ich habe mit Herrn Reatzo einen Nachmittag und zu Hause nochmals 4 Stunden investiert eure App zu Deployen. Es war nicht euer Fehler, aber "vergleichsweise reibungslos" ist ziemliche untertrieben. Gut möglich, dass Sie davon nichts mitbekommen haben.

Ich habe gelernt, wie wichtig **Linting** und **Code-Formatierung** für die Lesbarkeit und Wartbarkeit des Codes sind. Ausserdem kann ich nun die Applikation in AWS deployen, was ein grosser praktischer Fortschritt ist.

**Dieses Thema vom Sprint ist besonders relevant? Begründen Sie.**
Für mich ist **Linting und Formatierung** besonders relevant. Im Arbeitsalltag ist es oft schwierig, unformatierten Code zu lesen, was viel Zeit kostet. Linting verhindert ausserdem, dass Fehler unbemerkt in die Produktion gelangen.
Copy link
Contributor

Choose a reason for hiding this comment

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

Alle Fehler?

**Was sollte geändert werden?**
Einige Punkte könnten optimiert werden:
1. **Kürzere Vorträge**: Die Einführungen könnten kompakter gehalten werden, um schneller ins praktische Arbeiten einzusteigen.
2. **Docker-Einführung**: Das Erstellen des Dockerfiles hat viel Zeit gekostet, teilweise wegen unterschiedlicher Einstellungen. Es wäre hilfreich, hier weniger Zeit zu investieren oder eine simplere Lösung anzubieten, da Docker nicht immer bei jedem reibungslos funktioniert.
Copy link
Contributor

Choose a reason for hiding this comment

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

Welchen Dockerfiles? Vom Devcontainer oder vom Dockerfile fürs Deployment? Den Devcontainer werde ich zeimlich sicher optional halten in Zukunft, Dockerfiles fürs Deployment nach AWS sind unverzichtbar.

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.

2 participants