Closed Terra854 closed 7 years ago
Thanks for these changes. Though i am afraid i will not merge them like this. You need to split them up at least into multiple commits so individual changes become visible. Some of the changes are opinionated or do not make much sense to me. If you want me to merge some of this, please split things up into individual merge request, or if you feel this is more like all or nothing then at least into multiple commits. Thanks!
Thanks for the tip. I will make the changes
Do not close pull requests when you plan to update it to avoid loosing previous comments. Simply update/rebase/squash the existing branch and force push. The pull request automatically updates to the HEAD of its branch.
You mean i can reverse a commit?
Signed-off-by: Ben Tinner bentinner@yahoo.com.sg