search

htw-imi-showtime / showtime-website

The official IMI showtime website
10 stars 104 forks source link

M2 - Added pictures #349

Closed gerritbehrens closed 4 months ago

bkleinen commented 4 months ago

Danke, Sie haben einen Bug in meinem Prüfskript gefunden!

dennoch enthält der PR zuviele dateien und der Title stimmt auch nciht (siehe https://github.com/htw-imi-showtime/showtime-website/blob/main/doc/pull_request.md

--- DIFF: --- content/ss24/master/m1_gaming_for_future/Nabu.png content/ss24/master/m1_gaming_for_future/_index.md content/ss24/master/m2-food4future/_index.md content/ss24/master/m2-food4future/team_diro.jpg content/ss24/master/m2-food4future/team_gerrit.jpg content/ss24/master/m2-food4future/team_hasan.jpg content/ss24/master/m2-food4future/team_yosua.jpg showtime-website

bkleinen commented 4 months ago

wie gesagt, ich lerne selbst am meisten über git :/

laut meinem log (lokal) ist die base ihres pr a21967a,

da wäre folgender diff richtig (s.u.) - showtime-website git diff --name-only f74fb55 a21967a content/ss24/master/m2-food4future/_index.md content/ss24/master/m2-food4future/team_diro.jpg content/ss24/master/m2-food4future/team_gerrit.jpg content/ss24/master/m2-food4future/team_hasan.jpg content/ss24/master/m2-food4future/team_yosua.jpg showtime-website

-github gibt mir aber als base head of main und damit im diff auch die schon im main befindlichen Änderungen in m1.

bkleinen commented 4 months ago

die anzeige in changed files im PR ist hingegen korrekt:

image

gerritbehrens commented 4 months ago

Muss ich jetzt in irgendeiner Form tätig werden? Oder wie darf ich ihre Nachrichten verstehen? Ich habe eigentlich nur die Bilder hinzugefügt und diese im _index.md eingebunden.

bkleinen commented 4 months ago

und Sie haben ein neues submodul mit dem namen showtime-website angelegt. Da habe ich gestern eine gute Stunde nach gesucht, um diesen komischen Fehler zu finden, dass in den actions auf einmal das checkout-cleanup nicht mehr funktioniert hat.

Dank des falschen PR-Namens (siehe oben) habe ich einen bug in meinem Script gefunden.

s.o: dennoch enthält der PR zuviele dateien und der Title stimmt auch nciht (siehe https://github.com/htw-imi-showtime/showtime-website/blob/main/doc/pull_request.md

wenn Sie die beiden Sachen korrigieren, nehme ich den PR noch mit auf.