Gemfile Best Practices

gemfile_best_practices

source 'https://rubygems.org'

gem 'active_model_serializers', git: 'git://github.com/rails-api/active_model_serializers.git'
gem 'ember-rails', git: 'git://github.com/emberjs/ember-rails.git' # so we get the pre version
gem 'rack-mini-profiler', git: 'git://github.com/SamSaffron/MiniProfiler'
gem 'vestal_versions', git: 'git://github.com/zhangyuan/vestal_versions'

gem 'message_bus', path: 'vendor/gems/message_bus'
gem 'rails_multisite', path: 'vendor/gems/rails_multisite'
gem 'simple_handlebars_rails', path: 'vendor/gems/simple_handlebars_rails'

gem 'activerecord-postgres-hstore'
gem 'acts_as_paranoid'
gem 'airbrake', '3.1.2' # errbit is broken with 3.1.3 for now
gem 'clockwork', require: false
gem 'em-redis'
gem 'eventmachine'
gem 'fast_xs'
gem 'fastimage'
gem 'fog', require: false
gem 'has_ip_address'
gem 'hiredis'
gem 'hpricot'
gem 'i18n-js'
gem 'jquery-rails'
gem 'koala', require: false
gem 'multi_json'
gem 'mustache'
gem 'nokogiri'
gem 'oauth', require: false
gem 'oj'
gem 'pbkdf2'
gem 'pg'
gem 'rails'
gem 'rake'
gem 'redis'
gem 'redis-rails'
gem 'rest-client'
gem 'rinku'
gem 'ruby-openid', require: 'openid'
gem 'sanitize'
gem 'sass'
gem 'seed-fu'
gem 'sidekiq'
gem 'sinatra', require: nil
gem 'slim'  # required for sidekiq-web
gem 'therubyracer', require: 'v8'
gem 'thin'

# Gem that enables support for plugins. It is required
gem 'discourse_plugin', path: 'vendor/gems/discourse_plugin'

# Discourse Plugins (optional)
# Polls and Tasks have been disabled for launch, we need think all sorts of stuff through before adding them back in
#   biggest concern is core support for custom sort orders, but there is also styling that just gets mishmashed into our core theme. 
# gem 'discourse_poll', path: 'vendor/gems/discourse_poll'
gem 'discourse_emoji', path: 'vendor/gems/discourse_emoji'
# gem 'discourse_task', path: 'vendor/gems/discourse_task'

# Gems used only for assets and not required
# in production environments by default.
# allow everywhere for now cause we are allowing asset debugging in prd
group :assets do
  gem 'coffee-rails'
  gem 'coffee-script'  # need this to compile coffee on the fly 
  gem 'sass'
  gem 'sass-rails'
  gem 'turbo-sprockets-rails3'
  gem 'uglifier'
end

group :test, :development do
  gem 'certified'
  gem 'fabrication'
  gem 'guard-jasmine'
  gem 'guard-rspec' 
  gem 'guard-spork'
  gem 'image_optim'
  gem 'jasminerice'
  gem 'mocha', require: false
  gem 'rb-fsevent'
  gem 'rb-inotify', '~> 0.8.8', require: RUBY_PLATFORM.include?('linux') && 'rb-inotify'
  gem 'rspec-rails'
  gem 'shoulda'
  gem 'simplecov', require: false
  gem 'terminal-notifier-guard', require: RUBY_PLATFORM.include?('darwin') && 'terminal-notifier-guard'
end

group :development do 
  gem 'better_errors'
  gem 'binding_of_caller' # I tried adding this and got an occational crash
  gem 'pry-rails'
end

Wouldn't it also be a best practice to specify version requirements on your gems? You don't even have a version specified for the rails gem.