Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

Update dependencies (gems / Rails / Ruby) #1329

Closed
1 task
orenyk opened this issue Oct 25, 2015 · 12 comments · Fixed by #1367
Closed
1 task

Update dependencies (gems / Rails / Ruby) #1329

orenyk opened this issue Oct 25, 2015 · 12 comments · Fixed by #1367

Comments

@orenyk
Copy link
Contributor

orenyk commented Oct 25, 2015

We're currently one minor version behind (Rails v4.1 vs Rails v4.2), but with the pending release of Rails 5 we might just want to skip ahead after a few patch updates for stability. Any thoughts?

Update 2015/11/22

I'm bundling in all of our dependency updates into this issue, including our gem dependencies, Rails (to v4.2.x), and Ruby (to v2.2.x). This is targeted for a v6.0.0 release.

Update 2015/11/30

  • Update configuration documentation / Heroku buildpacks to reflect new static asset configuration variable
@orenyk
Copy link
Contributor Author

orenyk commented Nov 23, 2015

I've started reading through the upgrade guide; there doesn't appear to be too much to change, hopefully. I think if we include an update to the Ruby version as well (while we're updating dependencies) then maybe we'll bump our next release to be a major one (e.g. v6.0.0) with all dependencies updated. Thoughts?

@orenyk
Copy link
Contributor Author

orenyk commented Nov 23, 2015

After some more reading, this might present a problem but ties nicely into #1328, so I think that fits in with a major release as well 😄.

@orenyk orenyk changed the title Update Rails version Update dependencies (gems / Rails / Ruby) Nov 23, 2015
@orenyk orenyk modified the milestones: 5.6.0, Wish List Nov 23, 2015
@orenyk orenyk self-assigned this Nov 30, 2015
@orenyk orenyk mentioned this issue Nov 30, 2015
3 tasks
@orenyk
Copy link
Contributor Author

orenyk commented Nov 30, 2015

Ok, updating Ruby didn't appear to break anything 😄. Working on Rails update now to 4.2.5 (had to update Devise simultaneously for dependency reasons).

@orenyk
Copy link
Contributor Author

orenyk commented Dec 1, 2015

Ok, we're going to switch our SERVE_STATIC environment variable with RAILS_SERVE_STATIC_FILES as per the new default production.rb.

@orenyk
Copy link
Contributor Author

orenyk commented Dec 1, 2015

Welp, over 200 failed tests 😞

@orenyk
Copy link
Contributor Author

orenyk commented Dec 1, 2015

ok, most of that had to do with a deprecation in our factory. We still have a bunch of other errors, working through them now

@orenyk
Copy link
Contributor Author

orenyk commented Dec 1, 2015

Ok, dealt with most of the issues, a lot of it had to do with some of the new HTML sanitizers. We're still getting weird errors on "serialized_attributes" deprecations that I have to look into.

Also, one of our UserMailer specs is failing, but I'm actually confused as to how it passed previously - it has to do with our environment variable-triggered logger, using a Mailer Observer (#1127). As far as I can tell, the env_wrapper spec helper doesn't actually reload the Rails environment, so that observer isn't being registered for any test. I'm suspecting that our passing spec has to do with some other logging behavior that is no longer occurring in Rails 4.2.5. I'll have to keep investigating tomorrow...

@orenyk
Copy link
Contributor Author

orenyk commented Dec 8, 2015

So it turns out that we can't test configuration-dependent initializer functionality in RSpec, so I'm going modify the spec so that we always create the Mailer Observer and it only writes to the log if the ENV variable is set.

@orenyk
Copy link
Contributor Author

orenyk commented Dec 8, 2015

Alright, green build! We still have deprecation warnings about serialized_attributes but I think that might be from one of our gems so I'll try updating those and see if it resolves them.

DEPRECATION WARNING: `serialized_attributes` is deprecated without replacement, and will be removed in Rails 5.0. (called from realtime at /home/oren/.rbenv/versions/2.2.3/lib/ruby/2.2.0/benchmark.rb:303)
DEPRECATION WARNING: `serialized_attributes` is deprecated without replacement, and will be removed in Rails 5.0. (called from instance_eval at (eval):8)

@orenyk
Copy link
Contributor Author

orenyk commented Dec 9, 2015

the latest version of permanent_records, v4.0.0, doesn't install due to a dependency issue (see here), so I'm leaving it for now (Gemnasium doesn't say that it's urgent).

@orenyk
Copy link
Contributor Author

orenyk commented Dec 9, 2015

Ok, all dependencies updated, we have a green build and everything should be good to go! Opening PR now.

orenyk added a commit that referenced this issue Jan 4, 2016
Resolves #1329
- update Ruby to 2.2.3
- update Rails to 4.2.5
- update remaining gems to latest versions
orenyk added a commit that referenced this issue Jan 5, 2016
Resolves #1329
- update Ruby to 2.2.3
- update Rails to 4.2.5
- update remaining gems to latest versions
orenyk added a commit that referenced this issue Jan 12, 2016
Resolves #1329
- update Ruby to 2.2.3
- update Rails to 4.2.5
- update remaining gems to latest versions
orenyk added a commit that referenced this issue Jan 12, 2016
Resolves #1329
- update Ruby to 2.2.3
- update Rails to 4.2.5
- update remaining gems to latest versions
orenyk added a commit that referenced this issue Jan 12, 2016
@orenyk
Copy link
Contributor Author

orenyk commented Jan 13, 2016

Whoops, forgot to update app.json with the new ENV variable name, I'll open a quick issue and update the wiki as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant