Ston3s / plugin.video.xstream

Video Addon for XBMC
GNU General Public License v3.0
10 stars 8 forks source link

Patch 2 #96

Closed anis3 closed 8 years ago

Ston3s commented 8 years ago

Hallo anis3,

leider kann ich deine Anpassungen nicht so nicht über diese PR in die nightly und beta3 übernehmen. Habe die Anpassung jetzt Manuel in der Beta3 und nightly übernommen! Für dich zum Verständnis, wenn du neue Site-Plugins grundsätzlich entwickeln möchtest macht es schon Sinn eine Branch aus der "Lynx187/master" zu erzeugen aber wenn du das Ziel hast einer bestimmten Branch hinzuzufügen willst, wie in unserem Fall für ein Update, sollte man die Branch aus der "update-2.3.0-beta3" erzeugen. Hintergrund ist Folgender, wie du unter den Commits in diesem PR sehen kannst sind alle Änderungen die ich bereits in Beta3 hinzugefügt habe erneut enthalten. Also somit überflüssig/zu viel. Wenn du jetzt die Branch von der Beta3 neu erzeugt hättest würden die nur der letzte tatsächlich wichtige Commit "update streamtausch" aufgeführt werden. Dann hätte ich die PR auch sofort integrieren können.

Aber ganz nebenbei, ich hab dich doch für meinen Fork freigeschaltet. du kannst genau wie die anderen direkt dort die Änderungen vornehmen! ;) Noch viel einfacher, weil so erst gar kein Pull Request nötig ist...