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

Лень течет в наших жилах, охлаждая наш пыл #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Na1ve
Copy link

@Na1ve Na1ve commented Oct 23, 2013

No description provided.

Na1ve added 3 commits October 23, 2013 23:07
Результат скриптятины
@azproduction
Copy link

🆗 не хватает модульности, не хватает JSDoc, некоторые функции странно названы function rels( w1, w2, wordList ) (не понятно их назначение по названию). Лучше шаблоны не хранить в JS-е это и не удобно и не красиво.

@Na1ve
Copy link
Author

Na1ve commented Nov 15, 2013

rels — сокращение от relations. Честно, не думал, что кто-то будет это читать, так как дедлайн пролюбил. Так что и не стал марафет наводить =/ А так там много некрасивостей: константы в коде, кодстайл рандомный, нет общего стиля при передаче переменных e.t.c

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