Hallo Harald, > Was würdet ihr machen, wenn ihr zu einem von euch auf Github eingestellten > Projekt ohne vorherige Ankündung von einem bisher unbekannten User einen Pull > Request bekommt?
Das erlebe ich sehr häufig. Ich freue mich dann erst einmal. Eine Vorankündigung habe ich nur ganz selten bekommen (< 10 mal). Die einen waren unsicher, die anderen wollten einen sehr arbeitsaufwändigen Umbau vornehmen und das erst besprechen. > Der Request führt mehrere thematisch unterschiedliche Änderungen > ein. Einen Teil davon möchte ich definitiv nicht übernehmen. Bei einem anderen > sehe ich prinzipiell die Notwendigkeit einer Änderung, aber es gibt mehrere > denkbare Optionen und ich bin noch unschlüssig. Der vom Sourcecode-Umfang her > größte Teil ist durchaus in Ordnung, müsste allerdings noch ein bisschen > überarbeitet werden, außerdem wurde bei der Dokumentation geschlampt. Der Pull > Request besteht aus einem einzelnen Commit mit allen Änderungen, ein > Cherry-Picking > im Git-Sinne ist nicht möglich. Ich würde den PR kommentieren mit folgenden Hinweisen. 0) Danken 1) Bitte um einen Commit pro thematischer Änderung 2) Beschreiben, was in einem Commit gemerged werden soll (der Teil der durch kommt) 3) Beschreiben, was hat noch Diskussionsbedarf; gebe die Commits vor, in den Commits kann dann einzeln diskutiert / abgelehnt werden. Das hat den Vorteil, dass man jemanden anlernt auch noch weitere PRs nutzbingend vorzubereiten. Wenn keine Antwort mehr folgt kannst Du die Änderungen als patch herunterladen und anpassen. Wenn ich bei Gentoo so eine Situation habe, sieht das z.B. so aus: # get patch wget -O - https://github.com/gentoo/gentoo/pull/NNNN.patch | git am Bei Bedarf vor dem | bearbeiten oder vor dem Commit: # edit the PR git rebase -i HEAD~1 Wenn Du magst, kannst Du den Autor anpassen: # edit authorship git commit --amend --author="Joe Smith <[email protected]>" =========== Wenn der Code absolut nicht verwendbar ist, die Idee aber gut war, kann man auch selbst einen Commit daraus bauen und im Kommentar einfügen: Suggested-by: Full Name <[email protected]> Methode von kernel.org Beste Grüße, -- Jonas Stein
