Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Git Homework #19

Open
wants to merge 8 commits into
base: make-me-friendly
Choose a base branch
from
Open

Conversation

AVor0n
Copy link

@AVor0n AVor0n commented Nov 17, 2023

Выполнил задание в соответсвии с readme:

  • Сделал rebase ветки make-me-friendly, подчистив историю:
    • Коммиты, начинающихся с Fix, переместил внутрь соответствующих коммитов по добавлению алгоритмов
    • Удалил файл trash.bin из коммита Add bubbleSort
    • Перенес коммит про gitignore в начало
  • Решил конфликт при мерже ветки merge-me в make-me-friendly

Copy link
Owner

@gooverdian gooverdian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Все хорошо, но теперь потерялось авторство коммитов.

Мне казалось, в предыдущем ПР я видел твою аватарку рядом с коммитами, но может, не заметил ее отсутствие там.

Так бывает, когда в git config указан email, не соответствующий email github.
Попробуй поправить email в config и добавить один любой коммит к этой ветке для проверки

@gooverdian gooverdian added done Все хорошо styling Стилевые замечания 6 Найдено и выполнено дополнительное задание labels Nov 19, 2023
@AVor0n
Copy link
Author

AVor0n commented Nov 19, 2023

Все хорошо, но теперь потерялось авторство коммитов.

Мне казалось, в предыдущем ПР я видел твою аватарку рядом с коммитами, но может, не заметил ее отсутствие там.

Да, это из-за того, что предыдущий PR я из дома делал, а этот уже с рабочего компа, где почта рабочая указана.
Докинул комит с домашнего ноута, на нем у меня и ssh-подпись коммитов настроена

@gooverdian
Copy link
Owner

Ты можешь указывать разные конфиги для разных репозиториев, без флага --global, ты можешь настроить на любом компьютере любую почту для отдельного репозитория (это просто информация)

@gooverdian gooverdian removed the styling Стилевые замечания label Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6 Найдено и выполнено дополнительное задание done Все хорошо
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants