search

FreifunkMD / site-ffmd

Freifunk Magdeburg specific Gluon configuration
Creative Commons Zero v1.0 Universal
2 stars 12 forks source link

Change MTU to 1312 #122

Closed penguineer closed 6 years ago

penguineer commented 6 years ago

For https://github.com/FreifunkMD/site-ffmd/issues/121

LeSpocky commented 5 years ago

Hier fehlt eindeutig die Begründung für die Änderung, sowohl im Ticket als auch in der Commit-Message. Man kann sich zwar durch ein halbes dutzend Tickets klicken, aber selbst da fehlt mir eine sinnvolle Begründung, warum 1312 und nicht 1280 und nicht 1300 oder irgendein anderer Wert. So lässt sich in Zukunft nur sehr schwer nachvollziehen, warum der aktuelle Wert auf genau 1312 steht.

LeSpocky commented 5 years ago

Hab doch noch was gefunden: https://github.com/FreifunkMD/site-ffmd/pull/74#issuecomment-412209935 – Aber wie gesagt, sowas muss auch mit in die Commit-Message!

penguineer commented 5 years ago

(Daumen runter wegen Tonfall. Ein freundlicher Hinweis hätte gereicht. Die Begründung lässt sich noch ins Changelog schreiben – da wird sich auch eher gelesen.)

penguineer commented 5 years ago

see #126