Command line flags should override config options

[twavv]

Currently, it looks like flags specified in config.json override flags specified on the command line.

node-pg-migrate/bin/node-pg-migrate

Lines 232 to 241 in 40069f1

	SCHEMA = json[schemaArg] || SCHEMA
	CREATE_SCHEMA = json[createSchemaArg] || CREATE_SCHEMA
	MIGRATIONS_DIR = json[migrationsDirArg] || MIGRATIONS_DIR
	MIGRATIONS_SCHEMA = json[migrationsSchemaArg] || MIGRATIONS_SCHEMA
	CREATE_MIGRATIONS_SCHEMA = json[createMigrationsSchemaArg] || CREATE_MIGRATIONS_SCHEMA
	MIGRATIONS_TABLE = json[migrationsTableArg] || MIGRATIONS_TABLE
	MIGRATIONS_FILE_LANGUAGE = json[migrationFileLanguageArg] || MIGRATIONS_FILE_LANGUAGE
	CHECK_ORDER = typeof json[checkOrderArg] !== 'undefined' ? json[checkOrderArg] : CHECK_ORDER
	IGNORE_PATTERN = json[ignorePatternArg] || IGNORE_PATTERN
	DECAMELIZE = typeof json[decamelizeArg] !== 'undefined' ? json[decamelizeArg] : DECAMELIZE

Since the command-line flag handling happens earlier, should probably be

SCHEMA = SCHEMA || json[schemaArg]
// etc.

[dolezel]

True, command line options should have precedence. How is it that I didn't notice? 🤔

[dolezel]

I had some time today to look at it, but it turns more complicated as yargs now defines default values for boolean flags. Which will not be possible to override.
I will need to investigate it...