ctag-fh-kiel / ctag-tbd

CTAG TBD >>to be determined<< an extendible open source Eurorack sound module
Other
141 stars 43 forks source link

APCpp Bugfix for Triggers via CV (AD trigger and on/off toggle for options) #8

Closed Visuelle-Musik closed 3 years ago

Visuelle-Musik commented 3 years ago

Hi Robert, I tested the announced bugfix for APCpp and it looks fine. all the best, Mathias

Visuelle-Musik commented 3 years ago

Hallo Robert, ich denke ich muss dann meine Logik neu sortieren, zum Verständnis kurz wie es jetzt ist und wie ich da hin gekommen bin:

-> Das war dann leider komplettes Chaos, darauf hin hast Du a) Das Built-Script in das Repo aufgenommen b) Ich meinen Fork gelöscht und komplett neu aufgesetzt (man kann immer nur einen Fork zu einem Repo machen!)

-> Aktuell sind das: APCpp (Bugfix, getestet) / VctrSnt (Bugfix, ungetestet) / Formantor (Neuentwicklung, unfertig) / ctag-tbd/components/vult (Verzeichnis unveränderliche Komponenten für VULT)

Die Frage ist jetzt halt, wie ich das reorganisieren sollte und dann leider auch, wie das mit Git/Github Mitteln geht. Eine Möglichkeit wäre ja, alles aus dem Masterbranch meiner Fork raus zu nehmen und meine Build-Sachen in den Dev-Branch zu stecken, oder? Ist das ein sinnvoller Ansatz?

Wie ich die Sachen in den Dev-Branch rein bekomme ist mir grundsätzlich klar. Nur wie ich im Masterbranch auf den Stand bei Dir "zurückrollen" kann, weiß ich nicht. Sofern das geht würde ich das Rollback machen und dann nur APCpp neu in Master einfügen, korrekt?

Ich kann das auch gerne nachlesen Habe z.B. das hier gefunden: https://stackoverflow.com/questions/9646167/clean-up-a-fork-and-restart-it-from-the-upstream Nur will ich nicht alles komplett "verstrubbeln", dann müsste ich nämlich die Fork erneut löschen und von vorne anfangen, wobei das auch eine Variante wäre die für mich Ok ist, wollte mich halt nur mit Dir abstimmen, wie wir das konzeptionell am Besten machen und es dann auch so weiterlaufen kann ;-) Arbeite jetzt übrigens mit einer Kombi aus Gitkraken und der Gitshell in Gitkraken, wenn es gezieltere Aktionen sind, das geht eigentlich prima.

Danke+Gruß, Mathias

Am So., 28. Feb. 2021 um 08:23 Uhr schrieb Robert Manzke < notifications@github.com>:

@ctag-fh-kiel requested changes on this pull request.

Hi Mathias, thanks for the update. Can you please start the merge request from a fresh branch on your fork? There are many files changed here, which are not part of your APCpp changes... Thanks much :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ctag-fh-kiel/ctag-tbd/pull/8#pullrequestreview-600229148, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA5BUK4ZRAVBJYQGQ2MSR3TBHVPDANCNFSM4YIGPXIA .

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Virenfrei. www.avast.com https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>