Skip to content

Fix config loading without source name#140

Draft
Mirochill wants to merge 1 commit into
dlt-hub:develfrom
Mirochill:fix-126-config-project-dir
Draft

Fix config loading without source name#140
Mirochill wants to merge 1 commit into
dlt-hub:develfrom
Mirochill:fix-126-config-project-dir

Conversation

@Mirochill

@Mirochill Mirochill commented May 25, 2026

Copy link
Copy Markdown

Summary

  • ignore unset CLI options when loading a config file so values like project_name from the file are preserved
  • merge config file values with provided CLI values explicitly, with provided CLI values taking precedence
  • report a clear Typer validation error when neither a source name nor project_name is available

Fixes #126
Fixes #137

Validation

  • git diff --check
  • inspected the changed config loading and CLI project directory guard
  • Not run locally

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.

Cannot run dlt-init-openapi with config file Generator fails for missing project_dir

1 participant