-
Notifications
You must be signed in to change notification settings - Fork 0
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
merge/validate feature/notifications branch #39
Comments
Ja, weil das schon gemerget wurde :) Einfach ein neues branch von master aus erstellen und da das jar reinpacken ;) |
der diff sind jetzt 55 commits. mittelgeil.. :) und PR nicht vergessen |
mache es eben für dich |
Wie kann der in den master gemergt sein, wenn der master 55 commits ahead ist? |
Das remote branch wurde gemerget und gelöscht. du hattest das alte (nicht gesquashte) noch und hast erneut das remote erstellt. hab aber schon alles für dich gemerged ;) |
Achso, ok. Ich halte Squashing für keine gute Idee, das würde ich gern morgen nochmal diskutieren. |
naja, wenn du auf einem Feature branch arbeitest, ist automatischer squash vorm mergen eigentlich standard. Wenn du was ändern willst, solltest du eh immer von master neu abbranchen und einen neuen PR erstellen, damit alle drüber gucken können :) |
der feature/notifications branch hatte remote gefehlt, ich habe ihn erstmal wieder hinzugefügt. Bitte überprüfen/ggf. einmergen
The text was updated successfully, but these errors were encountered: