-
Notifications
You must be signed in to change notification settings - Fork 10
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
minor clarification to setup instructions in README #63
base: master
Are you sure you want to change the base?
Conversation
Do we really need this? I thought the output from I bet that line gets lost because I display the full path name for the file. Maybe breaking the "Then to resume, run ..." instructions to a new line would make that clearer? |
Yeah, it's clear enough when you run it. It's just that editing of the database.yml (and any things you might need to do in postgres such as setting up an owner/password for the dbase and making sure the server is running) isn't a minor step, and when I'm looking at all the setup steps together, it's helpful to be reminded of that part of it. |
@ericgj: Those things are true if you don't have a working postgres installation already set up, but typically postgres should start at boot and be configured in a way that doesn't require you to run any postgresql related commands to make use of a setup script like this. Linking to a general tutorial on how to setup postgresql properly for use with Rails would probably be worthwhile (especially since we want to make it as easy as possible for less experienced folks to contribute), but giving specific reminders about making sure your server is running seems redundant. |
Will look for such a tutorial to link to - I think that would be helpful to have. I understand about not giving every last step in the README. My main point isn't so much about configuring postgres, it's that you do actually have to edit the database.yml and then re-run the rake setup, and it would be helpful to get a heads-up about that in the instructions. So even if it said
|
That seems reasonable. What do you think @jordanbyron? |
@ericgj I'd like to keep as much of the "setup" process in
Then we only have one place were we need to maintain the setup process. Sound good? |
@jordanbyron yes that's a great idea. Should I add to this pull request or a create new one? |
Go ahead and use this pull request 🎩 |
To make it clear you run
bundle exec rake setup
a second time after editing your database.yml.(Sorry for not filing a ticket before submitting this, but I thought it was pretty much a no-brainer).