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

WIP: Upgrade to Rails 4.1 #21

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

WIP: Upgrade to Rails 4.1 #21

wants to merge 39 commits into from

Conversation

ericboehs
Copy link
Member

I'm creating a PR to track my progress.

Here's what's left:

  • Maybe add these Gems: compass?, selectize_rails?, fog/carrierwave/mini_magick/s3_direct_upload?, nprogress?, has_scope?, responders?, rollout?, table_print?, brakeman?, bullet?, binding_of_caller?, better_errors?, lol_dba?, rails_best_practices?, traceroute?, dray-rails?, timecop?
  • Add QueueClassic
  • Raise error on forbidden attrs
  • Add Rack CanonicalHost
  • Add font_assets
  • Configure fonts in in all ENVs
  • Add slim
  • Add high_voltage
  • Add bootstrap
  • Add simple_form
  • Add will_paginate (bootstrap styles?)
  • Add rack-mini-profiler?
  • Add devise
  • Add switch_user
  • Add capybara, poltergeist, database_cleaner, factory_girl and possibly minitest-metadata
  • Configure rails 4.1 mail preview (determine if recip interceptor is needed)
  • Configure Mandrill
  • Add memcachier
  • Determine if QC patch is necessary still since we are setting DATABASE_URL in .env
  • Add retina_tag (Needs to be applied and test retina tag with cdnsumo -- it may need patched) (Apply with: curl -L http://git.io/KebUPw|patch -p1
  • Add rails_12factor? heroku?
  • Add Airbrake
  • Add Newrelic
  • Add authority (rollify?)

- Add `rake test:environments` which iterates over `test/environments`
  and runs each file in a separate ruby process (to isolate requires)
- Check presence of multiple files (.envrc, .env.example, bin/setup, etc)
- Check AR validation errors are logged
- Ensure presence of gems, bins and assets
- Ensure SSL required in production
Rails.application.load_tasks

Rake::Task['default'].clear
task default: 'test:all'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should be able to just run rake now. Or better yet autotest.

@@ -0,0 +1,8 @@
namespace :test do
desc "Run environment tests for each environment"
task environments: :environment do
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fancy new tests to test this rails-template. This does not get called with rake test:all (maybe it should?). Call rake test:environments separately to ensure all is well.

@ghost ghost assigned ericboehs Jan 23, 2014
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.

1 participant