Skip to content

Conversation

@Awesome158
Copy link
Contributor

with cleaned up commit history

@Awesome158
Copy link
Contributor Author

Awesome158 commented Aug 11, 2025

Я сослался не на тот pull request 🤦‍♂️

@gretmn102
Copy link
Collaborator

Я сослался не на тот pull request 🤦‍♂️

Ничего страшного: опыт — дело наживное. Ни в коем разе больше не создавай дополнительные пулл реквесты. Есть один пулл реквест, с ним и работаем. Проделай следующее:

  • git rebase -i origin/main
  • передвинь blog: fix broken link куда-то в начало (я про него совсем забыл 😓)
  • напротив всех коммитов проставь r (use commit, but edit the commit message)
  • поменяй #33 на #34
  • blog: fix broken link -> blog(docs-add-chapters): fix broken link (#34)
  • git push --force 👀

@Awesome158
Copy link
Contributor Author

Awesome158 commented Aug 11, 2025

Ни в коем разе больше не создавай дополнительные пулл реквесты

Извини, но я пробовал через rebase — это получилась просто каша из конфликтов и всего подряд. А переделать с нуля заняло 10 секунд. Иногда нужно ставить своё время выше «самого правильного» способа. Честно говоря, я вообще не вижу в этом особой проблемы.

@gretmn102 gretmn102 merged commit 9e46a6c into QSPFoundation:main Aug 11, 2025
1 check passed
gretmn102 pushed a commit that referenced this pull request Aug 11, 2025
gretmn102 pushed a commit that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants