freemint / tos.hyp

The tos.hyp tries to document all functions from TOS. It also has information about MagiC, N.AES, MyAES, Geneva, XaAES, oAESis and some emulators.
https://freemint.github.io/tos.hyp
GNU General Public License v2.0
13 stars 11 forks source link

PR #84

Closed pulsar122 closed 6 years ago

th-otto commented 6 years ago

Why do you open a PR, only to immediately merge and close it? That does not make much sense, just messes up the commit history. Miro did a lot of work to fix similar things in the freemint repo, please don't start the same here. Same applies to PR #83 from David.

pulsar122 commented 6 years ago

Wie soll ich es denn machen?

Ich mache die Änderungen in meinem Zweig, dann einen PR zu freemint/tos.hyp. Dann muss ich doch einen Merge machen oder nicht?

th-otto commented 6 years ago

Anstatt die Änderungen in deinem eigenen fork zu machen, könntest du das Repository von FreeMiNT clonen, und die Änderungen dort machen und pushen. Das würde dir auch ersparen den PR zu öffnen. Eigentlich bräuchtest du dann deienn eigenen fork auch nicht mehr (es sei denn du hast da noch andere Änderungen oder Branches die du behalten möchtest).

Eine andere Möglichkeit wäre, beim mergen vorher das popup auf "Rebase and merge" zu stellen.