Update on 11-metaprogramming-part-2.slim

[smurfolan]

Spelling mistake fix

[mitio]

Това е доста по-добре, прогресираш :)

Все още има какво да се подобри. Например, commit съобщението ти е твърде общо. По-добър вариант би бил "Fix a typo in lecture 11". Кой е промененият файл се вижда ясно от съдържанието на самия commit.

Същото се отнася и за името на branch-а. И то може да е 11-fix-typo, например. След края на промяната, този branch ще бъде изтрит.

Аз съм окей да merge-на този PR и в текущия му вид, но искам да се упражниш в rebase-ване, защото вероятно ще ти се налага занапред.

За тази цел, ще те помоля да смениш commit съобщението на предложеното от мен или някаква друга вариация, която ти измислиш. За това ще ти трябват две неща:

1. git commit --amend
2. git push --force origin fixing-errors...

Точка едно не е rebase-ване, но пак пренаписва историята. Затова е необходимо и да се направи force push. Google за повече информация, или питай тук, ако имаш въпроси.

След като направиш тези неща, пиши тук, за да погледна и, евентуално, да merge-на.

[smurfolan]

Нащраках някакви неща, но не знам дали съм променил нещо.

[mitio]

Така е по-добре, благодаря. Ще го приема. Дал съм ти и бонус точка.

Само една забележка имам в момента и тя ще остане да я имаш предвид занапред и това са имената на commit съобщенията. Следното не е добро име на съобщение:

Update some_file.ext

Spelling mistake fix

Далеч по-подходящо е само:

Spelling mistake fix

Същото важи и за второто съобщение. Заменяш:

Update on 10-introspection-part-2-metaprogramming-part-1.slim

Pointing out GC is not a Class but a Module.

С това:

GC is not a Class but a Module

И всичко щеше да е перфектно :) Имай го предвид в следващия ти pull request.

Благодаря за приноса!