Mismatch in name of config file when created by cli setup vs what cli migrate expects

[mcobzarenco]

First, 💯 for the project!

The refinery_cli creates a refinery.toml config file when running refinery setup -- however, refinery migrate expects the file to be called Refinery.toml

As a passing comment, in the cli readme, the examples use refinery_cli, but the executable seems to be called refinery when installing as suggested with cargo install refinery_cli.

[jxs]

The refinery_cli creates a refinery.toml config file when running refinery setup -- however, refinery migrate expects the file to be called Refinery.toml

yeah, that has been fixed in the master, both are now using refinery.toml: https://github.com/rust-db/refinery/blob/master/refinery_cli/src/cli.rs#L18 and https://github.com/rust-db/refinery/blob/master/refinery_cli/src/cli.rs#L18

As a passing comment, in the cli readme, the examples use refinery_cli, but the executable seems to be called refinery when installing as suggested with cargo install refinery_cli.

yeah you are right, want to submit a PR fixing it?
thanks!

[mcobzarenco]

yeah, that has been fixed in the master, both are now using refinery.toml

👍 looking forward to a release, but in the meantime I'll switch to installing from master, thanks!

yeah you are right, want to submit a PR fixing it?

Sure thing! (#85)

[jxs]

+1 looking forward to a release, but in the meantime I'll switch to installing from master, thanks!

yeah I plan too as soon as nightly is fixed and I have time to address other features
thanks