Skip to content

Fix pg db name argument#131

Open
NathanaelRea wants to merge 2 commits intomalloydata:mainfrom
NathanaelRea:fix-pg-db-name
Open

Fix pg db name argument#131
NathanaelRea wants to merge 2 commits intomalloydata:mainfrom
NathanaelRea:fix-pg-db-name

Conversation

@NathanaelRea
Copy link

@NathanaelRea NathanaelRea commented Jan 30, 2026

Two fixes:

  1. Since the long argument was --database-name, the variable was passed as databaseName, but the connection was looking for database, and not saving the name in the config.
  2. Root arguments use -d as --debug, which will eat any -d used for database connections. Therefore I moved both postgres and snowflake -d to -n for "database name"

edit: accidentally closed

commander saves the long arg in camelCase. It was using databaseName, which was not captured when saving the connection
-d is used in root args as --debug
@NathanaelRea NathanaelRea changed the title Fix pg db name Fix pg db name argument Jan 30, 2026
@NathanaelRea NathanaelRea reopened this Feb 2, 2026
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.

1 participant