Miretpl commented on code in PR #72859: URL: https://github.com/apache/airflow/pull/72859#discussion_r3995975371
########## airflow-core/newsfragments/72859.bugfix.rst: ########## @@ -0,0 +1 @@ +The API server now honors the ``[api] worker_timeout`` configuration setting. Review Comment: I think that we can delete this newsfragment. PR title should be enough here. ########## airflow-core/tests/unit/cli/commands/test_api_server_command.py: ########## @@ -31,6 +33,23 @@ console = Console(width=400, color_system="standard") +def test_uses_worker_timeout_config_as_cli_default(): Review Comment: Could you add this test case to the test class below to improve consistency? -- 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]
