Исправление EXL, (текущая версия) :
И ещё некоторые советы по предстоящей работе в Git/GitHub. Возможно будут полезны.
1.
$ git log --format=fuller --graph
* commit c32c98e667e352bd8f44dcab496050339d4af043 (HEAD -> test)
| Author: Sunderland93 <sunderland93@Debian-Gnome>
| AuthorDate: Sat Nov 17 16:01:51 2018 +0400
| Commit: Sunderland93 <sunderland93@Debian-Gnome>
| CommitDate: Sat Nov 17 16:01:51 2018 +0400
|
| Very WIP: xdg-shell support
|
Вот это <sunderland93@Debian-Gnome> не является нормальным ящиком. Из-за этого на GitHub'е твой коммит выглядит с двумя именами аля Sunderland93 authored and Sunderland93 committed.
Задай себе нормальный ящик в .gitconfig, который совпадает с тем, что используется у тебя на GitHub'е. Так будет понятно, что коммиты идут именно с твоего профиля, а заинтересованные в изменениях смогут с тобой связаться.
2.
Ты зачем-то закрыл свой прошлый PR (там у некоторых коммитов кстати как раз был нормальный ящик), и создал новый. Это делать совсем было не обязательно. Вот, например, ты захотел исправить ту ошибку сборки с двойным добавлением одинаковых объектников, которую я указал выше. Ты можешь пойти следующими путями:
2.1. Сделать изменения в файле, исправляющие ошибку сборки, затем сделать что-то вроде:
$ git add .
$ git commit -m "Fix build error"
$ git push origin master
Эти действия добавят ещё один коммит к твоему PR на GitHub'е.
2.2. Сделать изменения в файле, исправляющие ошибку сборки, затем сделать:
$ git add .
$ git commit --amend
$ git push -f origin master
Эти действия перезапишут старый коммит в твоём PR на новый с исправленной ошибкой сборки. ИМХО это самый верный вариант, который позволяет избежать кучи коммитов и сохранить атомарность изменений. Именно подобным вкупе с git rebase -i и нужно было воспользоваться ранее, а не закрывать старый PR, создав новый.
Исходная версия EXL, :
И ещё небольшое замечание и советы по предстоящей работе. Возможно будут полезны.
1.
$ git log --format=fuller --graph
* commit c32c98e667e352bd8f44dcab496050339d4af043 (HEAD -> test)
| Author: Sunderland93 <sunderland93@Debian-Gnome>
| AuthorDate: Sat Nov 17 16:01:51 2018 +0400
| Commit: Sunderland93 <sunderland93@Debian-Gnome>
| CommitDate: Sat Nov 17 16:01:51 2018 +0400
|
| Very WIP: xdg-shell support
|
Вот это <sunderland93@Debian-Gnome> не является нормальным ящиком. Из-за этого на GitHub'е твой коммит выглядит с двумя именами аля Sunderland93 authored and Sunderland93 committed.
Задай себе нормальный ящик в .gitconfig, который совпадает с тем, что используется у тебя на GitHub'е. Так будет понятно, что коммиты идут именно с твоего профиля, а заинтересованные в изменениях смогут с тобой связаться.
2.
Ты зачем-то закрыл свой прошлый PR (там у некоторых коммитов кстати как раз был нормальный ящик), и создал новый. Это делать совсем было не обязательно. Вот, например, ты захотел исправить ту ошибку сборки с двойным добавлением одинаковых объектников, которую я указал выше. Ты можешь пойти следующими путями:
2.1. Сделать изменения в файле, исправляющие ошибку сборки, затем сделать что-то вроде:
$ git add .
$ git commit -m "Fix build error"
$ git push origin master
Эти действия добавят ещё один коммит к твоему PR на GitHub'е.
2.2. Сделать изменения в файле, исправляющие ошибку сборки, затем сделать:
$ git add .
$ git commit --amend
$ git push -f origin master
Эти действия перезапишут старый коммит в твоём PR на новый с исправленной ошибкой сборки. ИМХО это самый верный вариант, который позволяет избежать кучи коммитов и сохранить атомарность изменений. Именно подобным вкупе с git rebase -i и нужно было воспользоваться ранее, а не закрывать старый PR, создав новый.