Skip to content
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

[Documentation] Small Typo #11

Open
donaldev opened this issue Apr 25, 2019 · 0 comments · Fixed by aldenquimby/flywaydb-cli#4
Open

[Documentation] Small Typo #11

donaldev opened this issue Apr 25, 2019 · 0 comments · Fixed by aldenquimby/flywaydb-cli#4

Comments

@donaldev
Copy link

donaldev commented Apr 25, 2019

Hi there,

I'm new to flywayDB and new to your plugin and spent the day trying to get this to work yesterday - I couldn't figure out what I was doing wrong.

In your first example :
"scripts": { "migrate": "flyway -X -configFile=conf/flyway.conf migrate" }
I believe it should be -configFiles=.
I know it's only small but for somebody new to the tool, all I was working off was that first example - I read the flywayDB docs in the end and found the correct flag.

Secondly, it might be useful to show a project directory structure example, here's mine after getting it working :
-App
--conf
---flyway.conf
--node_modules
--sql
--- all migrations here
--.gitignore
-- package.json
-- README.md

N.B make sure to set the flyway.locations flag in the config file :
flyway.locations=filesystem:./sql/ - the default is inside the node_modules/flywaydb-cli package, which wouldn't be added to source control.

Just a suggestion as it would have helped me out a lot.

Thanks for the work

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 a pull request may close this issue.

1 participant