Skip to content
This repository has been archived by the owner on Mar 21, 2019. It is now read-only.

Upgrade to Rails 4 #5

Closed
aaronlerch opened this issue Aug 13, 2014 · 13 comments
Closed

Upgrade to Rails 4 #5

aaronlerch opened this issue Aug 13, 2014 · 13 comments

Comments

@aaronlerch
Copy link

I've got a branch where I did an upgrade of the app to Rails 4. I don't know that you want to merge it into master directly, but if you create a branch for rails 4 I'd be happy to issue a pull request into that branch.

Or if you prefer to update the app to rails 4 by default (probably reasonably safe) then I'm happy to do a pull request directly to master.

Or of course you can access my repo and merge it in yourself any way you like. :)
Thanks!

@aaronlerch
Copy link
Author

Oops, here's my repo: https://github.com/aaronlerch/prelaunchr/tree/rails_4

@danielschwartz
Copy link
Contributor

Sorry about the delay in getting back to you. Will take a look over the next week or so and figure out what the implications would be. I'm all for upgrading to Rails 4, just don't want to leave a bunch of people in the dust. Might make sense to do a final tag with Rails 3 and then have master be Rails 4.

@aaronlerch
Copy link
Author

😎 no worries!

@aaronlerch
Copy link
Author

It's been about 3 weeks so I figured I'd circle back around and see how you want to handle this? Thanks!

@danielschwartz
Copy link
Contributor

@aaronlerch since there are no tests, and since we will still have to support this project, but rails 4 is a useful addition lets do this: let's merge your changes into a branch that is not master called rails_4. If you could create two PR's, one that merges your changes into a branch called rails_4 on our repo, and one to update the README on our master letting people know there is a Rails 4 version at that branch would be great.

@aaronlerch
Copy link
Author

Sounds like a plan!

@phamdt
Copy link
Contributor

phamdt commented Dec 19, 2014

i submitted a PR with some tests in case that helps with this one

@ckhatton
Copy link

Hi @danielschwartz
I know you are busy, but can someone commit these pull requests, and also start committing the other pull requests, they are starting to pile up. I would like the fork the project, and especially use the Rails4 branch. Please add me as a contributor if you do not have the time. Thanks
:octocat:

@danielschwartz
Copy link
Contributor

@ckhatton we've unfortunately abandoned supporting this project. We'd be happy for you to fork it to a separate repo and become the owner of it, as long as you remove our assets from it.

We just don't have time to support this specific project in addition to all our other open source endeavors. Sorry!

@ckhatton
Copy link

@danielschwartz Okay that is fine - I will fork it and see if I can take on the pull requests (which I have never done and not sure is possible to a forked repo) 🎱

@wardpenney
Copy link
Contributor

I've done it over at https://github.com/wardpenney/prelaunchr

I need to remove the assets, but it is Rails 4.2, Ruby 2.2.0

@ckhatton
Copy link

@wardpenney Noice! 👍

Are you able to also add the outstanding pull requests?

@irubnich
Copy link
Contributor

Done

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

No branches or pull requests

6 participants