Skip to content
This repository has been archived by the owner. It is now read-only.

Fix examples in lecture 03 #18

Merged
merged 1 commit into from Oct 15, 2013
Merged

Fix examples in lecture 03 #18

merged 1 commit into from Oct 15, 2013

Conversation

@dkeranov
Copy link
Contributor

dkeranov commented Oct 14, 2013

Промених примерите, така че да отговарят на style guide'a.

@mitio

This comment has been minimized.

Copy link
Member

mitio commented Oct 14, 2013

Супер, ще приема промяната. Даже вече съм ти дал бонус точка за нея :)

Преди това трябва да направиш малко промени, обаче.

Първо, @NikolayGenov пусна един pull request преди теб, който оправя unless-а. Тоест, искам да махнеш тази промяна от твоя commit.

Второто нещо е, че имаш един излишен commit, който не ми трябва – dkeranov/ruby-lectures@5951d1b.

Моят съвет за най-лесен начин да оправиш тези неща е да направиш git pull --rebase. Прочети малко за git rebase (git help pull) и питай, ако имаш проблеми. Rebase пренаписва историята. В следствие на това, когато направиш git push, ще получиш предупреждение от git. В този конкретен случай, никой няма да си базира промени на този бранч, т.е. е окей да --force-неш push-а. Тогава и този pull request ще се обнови.

@dkeranov

This comment has been minimized.

Copy link
Contributor Author

dkeranov commented Oct 14, 2013

Благодаря, пробвах с git pull --rebase обнови ми промените, но commit'a 5951d1b остана. Пробвах с git rebase -i HEAD~2, но 5951d1b не в списъка.

@mitio

This comment has been minimized.

Copy link
Member

mitio commented Oct 14, 2013

Това е добър повод да се позабавляваш с Git и да научиш каква е връзката между комитите в историята и как работят бранчовете, rebase и прочее. Трябва да се върнеш малко назад в истрята, просто.

@mitio

This comment has been minimized.

Copy link
Member

mitio commented Oct 15, 2013

Помъчи се да решиш проблема сам, dig around, както се казва. Пробвай interactive rebase, например и squash на commit-ите. Ако не се справиш, питай за помощ. Но само след като си се помъчил добре сам :)

@dkeranov

This comment has been minimized.

Copy link
Contributor Author

dkeranov commented Oct 15, 2013

да, вчера си поиграх доста и според мен досчупих нещата, но днес ще се поровя още.

@mitio

This comment has been minimized.

Copy link
Member

mitio commented Oct 15, 2013

Идеята e, че понеже можеш да пренаписваш историята в Git, (почти) всичко е поправимо. Простро трябва да grok-неш как работят нещата на ниско ниво (а там концепцията е доста проста) и да намериш подходящата комбинация от команди, която да ти позволи да извършиш съответната промяна в историята.

Сравнително прост и дървен начин е просто да се върнеш назад в историята, да дръпнеш промените от upstream и да приложиш твоите промени ръчно, или с cherry-pick. И след това push --force...

@dkeranov

This comment has been minimized.

Copy link
Contributor Author

dkeranov commented Oct 15, 2013

done, добро упражнение стана! :))

@mitio

This comment has been minimized.

Copy link
Member

mitio commented Oct 15, 2013

Perfecto! Благодаря! Merge-вам.

P.S. Би било полезно да споделиш пътя, по който си го оправил, за бъдещите поколения.

mitio added a commit that referenced this pull request Oct 15, 2013
@mitio mitio merged commit 2eb979d into fmi:master Oct 15, 2013
@dkeranov

This comment has been minimized.

Copy link
Contributor Author

dkeranov commented Oct 17, 2013

Да :), в случая проблема ми беше, че направих merge'a с --no-ff или по-точно

git checkout upstream_master
git pull
git checkout master
git merge --no-ff upstream_master
git push origin master
git checkout -b 03-fix-examples
git push origin 03-fix-examples

Хубаво е да се виждат правените merge commits в лога и графа, но в случая беше излишно.
За съжаление git rebase -i HEAD~2 не ми показа commit'a получил се при merge na master с upstream_master.
Всичко се оправи с git reset --hard, което някак си исках избегна. Вярно отидоха няколко часа, но пък съм доволен :)

git checkout master
git reset --hard <sha1_commit_id>
git push --force origin master
git checkout 03-fix-examples
git reset --hard <sha1_commit_id>
git push --force origin 03-fix-examples
@dkeranov dkeranov deleted the dkeranov:03-fix-examples branch Oct 17, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants
You can’t perform that action at this time.