Правописни грешка

[bostko]

No description provided.

[mitio]

@bostko, благодаря за корекциите! :)

Това е вторият PR, който ни пращаш и който не следва правилата ни за pull request-и. Преди беше предложил (резонно) да се махне .rvmrc файлът от проекта, но така и не отговори на нашите напътствия как искаме да се оформи PR-а.

Сега склонен ли си да направиш този pull request както трябва?

[bostko]

Мисля че не е добре да има правописни грешки в условието на задачата :)

[bostko]

Съгласен съм че за предния pull request трябваше да дам аргументация, но за този очевидно не е добре да има правописни грешки.

[mitio]

Напълно съм съгласен :) Определено искам да приема този pull request.

Въпросът е, че има определени правила, които се стеремим да налагаме за pull request-ите, приемани в проектите около курса по Руби, като го правим до голяма степен с педагогическа цел, зa:

• Да свикне човек, че да се клонира хранилище е тривиална, евтина и бърза операция
• Да свикне човек, че да се направи нов branch е също такава бърза и лесна операция
• Да свикне човек как се работи с инструментите на Git
• Да свикне човек с добрите практики при изпращане на pull request-и, като например че е добре да се види какъв е стилът на commit-съобщенията в проекта до момента и да се направи опит да се запази този стил (дали са на английски, в сегашно или минало време са, дали започват с главна буква и прочее)

Основната цел е след като е напрвил няколко pull request-а, да спре да възприема подсъзнателно действията, необходими за направата на един pull request, като бариера за това да направи PR за някой друг, произволен open source проект. Искаме тези действия да са навик, рутина, за да може човек да не мисли за тях, а в центъра на съзнанието му да е самата промяна, която иска да предложи. И това може да се постигне само с практика, дори и за най-тривиалните промени, каквито са смяна на три символа в readme-файла :)

Очевидният факт, че не е добре да има правописни грешки в условието на задачата няма нищо общо с тези разсъждения, струва ми се :)

Затова, склонен ли си да приемеш напътствия сега как да направиш този PR според нашите изисквания?

[mitio]

@bostko, сега разбирам – нямам предвид само аргументацията. Наистина, в този конкретен случай, аргументация не е нужна. Но има други неща:

• Трябва да е направен от topic branch, например hw03-fix-typos. След като си клонирал хранилището, става като направиш git checkout -b hw03-fix-typos
• Commit съобщението трябва да е на английски, за да спазва стила на останалите събощения в хранилището

Тези двете са достатъчни за този PR. Ако искаш да ги изпълниш, ще трябва:

1. Да затвориш този PR.
2. Да си клонираш хранилището с git clone ... някъде и да влезеш в тази папка.
3. Да направиш нов branch и да влезеш в него; това може да стане с една команда: git checkout -b hw03-fix-typos.
4. Да направиш корекциите си.
5. Да ги commit-неш с подходящо съобщение на английски.
6. Да push-неш от твоя branch в твоя fork: git push origin hw03-fix-typos
7. Да пуснеш нов pull request в GitHub (той ще ти предложи бърз бутон от горната част на хранилището, понеже ще се усети, че наскоро си push-нал в topic branch)
8. Да ми кажеш какъв ти е факултетния номер, ако си част от курса по Ruby. Иначе - скип дис степ :)

Чакам с нетърпение резултата! :)

[skanev]

Това е популярен рефакторинг, известен като "Replace a typo in file with a typo in a commit message". Определено е по-добро да местим дефектите от production кода в историята, но някак, трябва да можем и по-добре.

[mitio]

@bostko Вальо? Ще го правиш ли това?

[bostko]

Направих го, факултетният ми номер е 44585.

[mitio]

Екстра. Това вече можем да го затворим, оправено е в #23.