10000 Fixes https://github.com/progit/progit2/issues/1310 by spotlesscoder · Pull Request #28 · progit/progit2-de · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fixes https://github.com/progit/progit2/issues/1310 #28

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Oct 18, 2019
Merged

Fixes https://github.com/progit/progit2/issues/1310 #28

merged 3 commits into from
Oct 18, 2019

Conversation

spotlesscoder
Copy link
Contributor

No description provided.

@pastatopf
Copy link
Collaborator

Der Text wird eingefügt, sobald es im englischen Repository aufgenommen wurde.

Danke für die Mitarbeit :-)

@max123kl
Copy link
Collaborator

@CodingSpiderFox
Bitte den Text umformatieren, entsprechend Fix #1310 im engl. Repo
Danke

@spotlesscoder
Copy link
Contributor Author

Ich muss erstmal verstehen, wie da der Zielzustand aussehen soll :D

@max123kl
Copy link
Collaborator

sieh hier: progit/progit2@58a8f3f

@spotlesscoder
Copy link
Contributor Author

Achso, das bezog sich jetzt auf den anderen PR, stimmt... wie Du siehst bin ich gerade maximal verwirrt ... progit/progit2#1316

@max123kl
Copy link
Collaborator

Bitte halte dich bei der Formatierung an die engl. Version. Dort sind die Klammern gesetzt!
Nur wenn deren Formatierung sinnentstellend sein sollte, wollen wir davon abweichen.

@spotlesscoder
Copy link
Contributor Author

Habe bei der englischen Version einen PR mit den gleichen Änderungen erstellt progit/progit2#1317

@max123kl
Copy link
Collaborator

Dann halte bitte die Änderungen bitte auch getrennt.
Solange eine Änderung im englischen Repo nicht gemergt wurde, werden sie hier nicht einfließen können.

@spotlesscoder
Copy link
Contributor Author

OK

@spotlesscoder
Copy link
Contributor Author

Habe den commit aus diesem Branch entfernt, inzwischen würde progit/progit2#1317 aber auch gemerged

@max123kl
Copy link
Collaborator

So wie ich das hier sehe, wäre dieser PR (#28) überflüssig, da du einen neuen PR #35 nachgeschoben hast, der die Korrekturen hier beinhaltet.
Wenn das so richtig ist, dann schließe diesen PR hier und hinterlasse bei #35 einen entsprechenden Kommentar.
Zukünftig bitte ich dich, nicht Patches in mehreren PRs, quasi doppelt, anzuführen. Das macht, wie Ben schon im engl. Repo anmerkte, nur unnötigen "noise" in den Logs und erschwert das Reviewen.

@pastatopf pastatopf merged commit dc6f27b into progit:master Oct 18, 2019
@spotlesscoder
Copy link
Contributor Author
spotlesscoder commented Oct 18, 2019 via email

@max123kl
Copy link
Collaborator

OK
pastatopf hat ja inzwischen gemerged und auch deine Änderungen aus #35 eingefügt.
Für mich war das auch etwas verwirrend.

@CodingSpiderFox
Da du für deinen ersten Beitrag ausgerechnet die import-svn-Datei vorgenommen hast, gehe ich davon aus, das du einige Erfahrung mit SVN hast.

Wenn das so richtig ist, würde ich dich bitten, bei den beiden SVN-betreffenden Dateien aus Kapitel 9, meine Übersetzung zu reviewen um mögliche Ungenauigkeiten zu verbessern.

@spotlesscoder spotlesscoder deleted the patch-1 branch October 18, 2019 07:59
@spotlesscoder spotlesscoder restored the patch-1 branch October 18, 2019 07:59
@spotlesscoder
Copy link
Contributor Author

Habe in PR #38 ein paar Vorschläge eingearbeitet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@spotlesscoder @pastatopf @max123kl
0