uranusjr commented on code in PR #31415: URL: https://github.com/apache/airflow/pull/31415#discussion_r1199991835
########## airflow/config_templates/config.yml: ########## @@ -452,6 +452,14 @@ core: database: description: ~ options: + alembic_ini_file_path: + description: | + Optional path to a custom alembic.ini file. If provided, these configurations + will be prioritised over the default alembic.ini file. + version_added: 2.7.0 + type: string + example: ~ + default: ~ Review Comment: I wonder if it’s a good idea to make this a _mandatory_ value, defaulting to `alembic.ini`. If this is a relative path, we resolve it against the Airflow package dir (like we do right now), and if absolute it can be used directly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
