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

Fix hardcoded error #65

Merged
merged 1 commit into from Dec 10, 2013

Conversation

Projects
None yet
3 participants
@AngelVenchev
Contributor

AngelVenchev commented Dec 10, 2013

Fixes an error in resulting code on slide 32

@mitio

This comment has been minimized.

Member

mitio commented Dec 10, 2013

А, супер! Благодаря!

mitio added a commit that referenced this pull request Dec 10, 2013

@mitio mitio merged commit 1016e2b into fmi:master Dec 10, 2013

@ignisf

This comment has been minimized.

Member

ignisf commented Dec 10, 2013

Само малка забележка - неписаните ни правила диктуват имената на branch-овете да са в spinal case :)

@AngelVenchev

This comment has been minimized.

Contributor

AngelVenchev commented Dec 11, 2013

Мерси за забележката, дали може да ми дадете линк, да прочета тези правила (освен ако не става дума за style-guide-а)

On Tuesday, December 10, 2013 10:53 PM, Petko Bordjukov notifications@github.com wrote:

Само малка забележка - неписаните ни правила диктуват имената на branch-овете да са в spinal case :)

Reply to this email directly or view it on GitHub.

@mitio

This comment has been minimized.

Member

mitio commented Dec 11, 2013

За това конкретно правило, което Петко споменава, нямаме изписан документ. Обикновено, преди да направиш PR, прехвърляш вече направени такива и гледаш какви конвенции са спазвани в тези от тях, които са merge-нати. Може да видиш и как се кръщават branch-овете в самия проект, ако такива има. Задължително гледаш стила на commit съобщенията. Вадиш си изводите сам, с други думи :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.