Skip to content

added checks to see if tables already exist #62

Merged
webvimark merged 6 commits intowebvimark:masterfrom
mnglkhn:master
Jul 7, 2015
Merged

added checks to see if tables already exist #62
webvimark merged 6 commits intowebvimark:masterfrom
mnglkhn:master

Conversation

@mnglkhn
Copy link
Copy Markdown
Contributor

@mnglkhn mnglkhn commented Jun 30, 2015

No description provided.

@webvimark
Copy link
Copy Markdown
Owner

HI, thank you for suggestion, but why it should be checked ?

If you already run those migrations, then they will be not applied again.
And if you for some reason already have those tables in database, then they should fall and let you be warned and take some actions.

Do you see some scenario when it will be usefull ?

@mnglkhn
Copy link
Copy Markdown
Contributor Author

mnglkhn commented Jul 1, 2015

Hi,

My use case scenario was for when I would already have the user table and not be able to progress with the remaining migrations.

regards,

Sent from my iPad

On Jul 1, 2015, at 12:42 AM, webvimark notifications@github.com wrote:

HI, thank you for suggestion, but why it should be checked ?

If you already run those migrations, then they will be not applied again.
And if you for some reason already have those tables in database, then they should fall and let you be warned and take some actions.

Do you see some scenario when it will be usefull ?


Reply to this email directly or view it on GitHub.

@webvimark
Copy link
Copy Markdown
Owner

Sorry for slow replay. I think that it may help =)
Thank you!

webvimark added a commit that referenced this pull request Jul 7, 2015
added checks to see if tables already exist
@webvimark webvimark merged commit d2b5021 into webvimark:master Jul 7, 2015
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.

2 participants