evilzorge/ads

ads

ads's Issues

конфиги в репозитории

Сейчас у тебя в репозитории есть файл database.yml. Если я попытаюсь развернуть приложение у себя, то наткнусь на проблему: у меня могут стоять другие данные для доступа к БД, а если я их изменю, файл изменится, и уже ты не сможешь нормально с ним работать. Так не делается потому что:

1. Усложняет совместную разработку
2. Небезопасно, так как там могут оказаться пароли к продакшну, например.
   Как исправить: создаёшь папку config/examples, кладёшь в неё твои файлы с конфигами, но не с настоящии данными, текущие файлы с конфигами добавляешь в гитигнор и удаляешь из репозитория(но не локально). Так нужно поступать с любыми файлами, где хранятся уникальные пароли/логины/ключи доступа.

gitignore

в проекте есть файлы, содержащие абсолютные пути (например, к логам), логи и прочая информация, которая является уникальной для проекта на каждой отдельной машине. Пример:
https://github.com/EvilZorge/ads/blob/master/log%2Fdevelopment.sphinx.pid
https://github.com/EvilZorge/ads/blob/master/config%2Fdevelopment.sphinx.conf
Необходимо добавить нужные файлы/папки в gitignore, чтобы не возникало конфликтов при разворачивании проекта кем-нибудь другим.

Описание коммитов

Старайся давать логически понятные понятные имена коммитам. Не всегда ясно, что такое "some fixes" и где что было исправлено. Пиши что-то вроде "resolved/fixed problem with smth in $where"
http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html

Комментарии в файлах.

В некоторых файлах есть закоментированный код. Комментарии должны нести смысловую нагрузку, и если какой-то код закоментирован, нужно пояснять, почему так. Если он не нужен- удалять.
Убрать закоментированный код из всех файлов, также почистить файлы вроде routes.rb и gemfile.

Файлы-пустышки

В репозитории есть файлы-пустышки(хелперы, стили и т.д), которые генерируются автоматически, но никак не используются, например:
https://github.com/EvilZorge/ads/blob/master/app/assets/stylesheets/admin/types.css.scss
https://github.com/EvilZorge/ads/blob/master/app/assets/javascripts/advertisments.js.coffee
Нужно удалить все эти файлы и добавлять вручную по мере необходимости, а также отключить их автогенерацию в конфиге.

.ruby-version и .ruby-gemset

В корне приложения отсутствуют файлы .ruby-version и .ruby-gemset. Эти файлы нужны, чтобы у разработчиков не возникало проблем из-за разных версий руби. Их можно сгенерировать через rbenv/rmv или добавить самому.

ability

Переписать абилити, используя метод send
http://ruby-doc.org/core-2.0/Object.html#method-i-send