Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update the source code layout #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wireless-ghost
Copy link

These changes are the latest updates in Source Code Layout part from ruby style guide (https://github.com/bbatsov/ruby-style-guide) by @bbatsov.

@mitio
Copy link
Member

mitio commented Oct 16, 2014

Това всички разлики между двете неща ли са, или е само частта за Source Code Layout?

И нещо не е наред с тях, понеже виждам, че има коментари от вида #bad, тоест, без интервал след диеза, а трябва да има интервал там и в оригинала има.

Защо избра само тази част?

@wireless-ghost
Copy link
Author

Всичките разлики са - да. Имаше само едно нещо, което не съм променил, защото не бях сигурен как трябва стане. Тук ли трябва да го посоча или да направя issue?
Относно #bad - бях ги объркал и мислех, че съм ги оправил - ще го прегледам пак и ще ги изчистя. Нов PR ли трябва да направя?
Само тази част е защото започнах по ред. Source Code Layout е първата. Ако е по-добре всичко на един път - ще прегледам разликите за всички части и ще направя нов PR.

@mitio
Copy link
Member

mitio commented Oct 22, 2014

По-добре изчакай с новия pull request. Все още обмислям как да процедираме с нашето ръководство – дали да го обновяваме, или да се осланяме само към community ръководството, като евентуално единствено опишем изключенията.

Ще помисля още малко и ще решим. Засега нека оставим текущия PR да виси и да чака.

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