search

yddaPyo / Info-Klausur

Info klausur sven, patrick, arthur pascal
Creative Commons Zero v1.0 Universal
1 stars 2 forks source link

.gitignore & HelloWorld (Kotlin/Maven) hinzugefügt #3

Closed archdtd111 closed 7 years ago

archdtd111 commented 7 years ago

Keine Ahnung obs richtig ist oder nicht. Ich war verbissen darauf jetzt noch was zu arbeiten. Die .gitignore's habe ich mir ausm Internet gezogen und die Maven/Kotlin-Datei habe ich einfach mal erstellt. Musste bisschen was in den Settings ändern damit das lief. Schaut es euch mal an. Hab auch einfach irgendwas an der pom.xml gemacht...es ist spät und meine Birne platzt :D

archdtd111 commented 7 years ago

Hallo Nicolai,

habe jetzt an ein zwei Sachen gearbeitet und auf meinen Fork hochgeladen. Wollte jetzt einen neuen PR machen um quasi meinen Fork (HEAD) mit dem
Base Fork (yddaPyo/Info-Klausur) zu mergen. Aber da wird mir keine Möglichkeit angezeigt den PR zu machen. Also
nicht mal ein Button oder so ähnliches. Im Anhang ein Screenshot.

Mit freundlichen Grüßen,

Artur \,,/

nipafx commented 7 years ago

Die Nachricht hat keinen Anhang. Falls du sie als E-Mail verfasst hast, ging die an GitHub und der Anhang wird dann (anscheinend) ignoriert.

nipafx commented 7 years ago

Ok, hab den Anhang in der Mail bekommen. Ich füge ihn mal hier ein.

:bulb: Wenn du einen Kommentar verfasst, steht am unteren Ende wie man Dateien anhängt.

fuernicolai .

nipafx commented 7 years ago

habe jetzt an ein zwei Sachen gearbeitet und auf meinen Fork hochgeladen.

Hier siehst du welche Commits in deinem Fork auf master sind - der jüngste ist fdd01584.

Wollte jetzt einen neuen PR machen um quasi meinen Fork (HEAD) mit dem Base Fork (yddaPyo/Info-Klausur) zu mergen.

Man kann keine Forks mergen, sondern nur Branches. Ich vermute, dass du das meinst - ist aber wichtig, dass du die richtigen Worte benutzt.

Aber da wird mir keine Möglichkeit angezeigt den PR zu machen.

Sieht für mich so aus als würde GitHub dich darauf hinweisen, dass es bereits einen Pull Request zwischen den beiden Branches gibt. Stimmt auch, diesen hier.

Bestehende PRs kannst du einfach erweitern, indem du weitere Commits auf den Source Branch pusht. Und das ist auch schon passiert: Guck mal direkt über deinem Kommentar, auf den ich hier antworte. Da taucht genau der jüngste Commit (fdd0158) auf.

nipafx commented 7 years ago

Damit die zugeklappte Nachricht nicht untergeht hier ein Link dorthin.

archdtd111 commented 7 years ago

Ich habe es schon so gemeint wie ich es ausgedrückt habe. Mein Problem ist, ich habe meinen Fork aktualisiert, dh, dort sind alle Neuerungen die ich lokal gemacht habe schon drauf gepusht. Habe alle Änderungen lokal über den Master gemacht. Also wenn ich jetzt von meiner Lokalen auf das usprüngliche Projek upstreame, dann wird es ja automatisch aktualisiert und ich kann das nicht über einen PR managen.

Greetings,

Artur

Am 2017-08-03 20:18, schrieb Nicolai Parlog:

habe jetzt an ein zwei Sachen gearbeitet und auf meinen Fork hochgeladen.

Hier [1] siehst du welche Commits in deinem Fork auf master sind - der jüngste ist fdd0158 [2].

Wollte jetzt einen neuen PR machen um quasi meinen Fork (HEAD) mit dem Base Fork (yddaPyo/Info-Klausur) zu mergen.

Man kann keine Forks mergen, sondern nur Branches. Ich vermute, dass du das meinst - ist aber wichtig, dass du die richtigen Worte benutzt.

Aber da wird mir keine Möglichkeit angezeigt den PR zu machen.

Sieht für mich so aus als würde GitHub dich darauf hinweisen, dass es bereits einen Pull Request zwischen den beiden Branches gibt. Stimmt auch, diesen hier.

Bestehende PRs kannst du einfach erweitern, indem du weitere Commits auf den Source Branch pusht. Und das ist auch schon passiert: Guck mal direkt über deinem Kommentar, auf den ich hier antworte. Da taucht genau der jüngste Commit (fdd015) auf.

-- You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub [3], or mute the thread [4].

*

Links:

[1] https://github.com/archdtd111/Info-Klausur/commits/master [2] https://github.com/yddaPyo/Info-Klausur/commit/fdd015845a2c69211c89ee3267983e355b98449b [3] https://github.com/yddaPyo/Info-Klausur/pull/3#issuecomment-320049511 [4] https://github.com/notifications/unsubscribe-auth/AdIrOOOgQCrdZUo2UZAQByKn8JGF_o0gks5sUg8BgaJpZM4OrvnQ

nipafx commented 7 years ago

Ich habe es schon so gemeint wie ich es ausgedrückt habe.

Beziehst du dich hierauf?

wollte jetzt einen neuen PR machen um quasi meinen Fork (HEAD) mit dem Base Fork (yddaPyo/Info-Klausur) zu mergen

Das geht nicht, man kann keine Forks mergen. Die Konzepte Fork (ein ganzes Git Repository mit Branches, Tags, etc) und Merge (eine Aktivität zwischen zwei Branches) ergeben zusammen keinen Sinn.

Also wenn ich jetzt von meiner Lokalen auf das usprüngliche Projek upstreame, dann wird es ja automatisch aktualisiert und ich kann das nicht über einen PR managen.

Wenn du mit "ursprüngliches Projekt" dieses hier und nicht deinen Form meinst, ist das richtig. Deswegen sieht der normale Arbeitsprozess vor, ein Projekt zu forken und dann den Fork zu klonen, so dass der Fork als origin Remote verwendet wird. Zusätzlich kann man dann noch das Originalprojekt als upstream Remote hinzufügen, aber davon pullt man üblicherweise nur.

Was gibt denn git remote -v aus?

nipafx commented 7 years ago

Vielleicht ist es einfacher noch einmal von vorne anzufangen. Du könntest das hier machen:

nipafx commented 7 years ago

Ich glaube #5 und #10 machen diesen Pull Request unnötig und ihr könnt ihn schließen (nicht mergen!).