[GitHub] [airflow] o-nikolas commented on a diff in pull request #27974: [AIP-51] Add helper to import default executor class

[GitBox <gi...@apache.org>]

o-nikolas commented on code in PR #27974:
URL: https://github.com/apache/airflow/pull/27974#discussion_r1035362093


##########
airflow/executors/executor_loader.py:
##########
@@ -65,18 +65,23 @@ class ExecutorLoader:
         DEBUG_EXECUTOR: "airflow.executors.debug_executor.DebugExecutor",
     }
 
+    @classmethod
+    def get_default_executor_name(cls) -> str:
+        """Returns the default executor name from Airflow configuration

Review Comment:
   If you view the rest of the module, all the docs use the plural verb form (`Loads the executor...`, `Creates a new instance of...`) so I will keep that form here. But I will add the missing period! 



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org