monterail / guidelines

[DEPRECATED] We are Ruby on Rails experts from Poland. Think hussars. Solid & winged. These are our guidelines.
71 stars 17 forks source link

Add schema.rb to gitignore #187

Closed teamon closed 11 years ago

teamon commented 11 years ago

Why:

/cc @jandudulski, @sheerun, @chytreg, @szajbus, @Ostrzy, @jcieslar, @tallica

sheerun commented 11 years ago

ok

jandudulski commented 11 years ago

Hm, deployment scripts doesn't use them?

teamon commented 11 years ago

Deployment scripts run rake db:migrate.

jandudulski commented 11 years ago

Ok than.

jcieslar commented 11 years ago

:+1:

sheerun commented 11 years ago

Don't CI run db:schema:load instead of running migrations?

szajbus commented 11 years ago

Ok, let's igonore. If the world explodes, we'll commit it back.

teamon commented 11 years ago

@sheerun Can you confirm that?

sheerun commented 11 years ago

I don't need to confirm. It is definition of this task.

On Fri, Oct 25, 2013 at 1:27 AM, Tymon Tobolski notifications@github.comwrote:

@sheerun https://github.com/sheerun Can you confirm that?

— Reply to this email directly or view it on GitHubhttps://github.com/monterail/guidelines/issues/187#issuecomment-27042707 .

teamon commented 11 years ago
rake db:schema:load              # Load a schema.rb file into the database

This does not mention CI at all.

szajbus commented 11 years ago

So I'm commiting the change since we all agree.

Sorry for commiting directly to master, I forgot to change the branch in Github inteface.

sheerun commented 11 years ago

I thought you meant something else.

On Fri, Oct 25, 2013 at 1:05 PM, Tymon Tobolski notifications@github.comwrote:

rake db:schema:load # Load a schema.rb file into the database

This does not mention CI at all.

— Reply to this email directly or view it on GitHubhttps://github.com/monterail/guidelines/issues/187#issuecomment-27082089 .