Fixing static checks for FabAirflowSecurityManagerOverride (#33416)

Fixing static checks. FabAirflowSecurityManagerOverrideDb and FabAirflowSecurityManagerOverrideOauth were failing pydocstyle rule D205
apache · Aug 15, 2023 · cc360b7 · cc360b7
1 parent 3766ab0
commit cc360b7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/airflow/auth/managers/fab/security_manager/modules/db.py b/airflow/auth/managers/fab/security_manager/modules/db.py
@@ -34,16 +34,16 @@
 
 class FabAirflowSecurityManagerOverrideDb:
     """
+    FabAirflowSecurityManagerOverride is split into multiple classes to avoid having one massive class.
+
     This class contains all methods in
     airflow.auth.managers.fab.security_manager.override.FabAirflowSecurityManagerOverride related to the
     database.
 
-    FabAirflowSecurityManagerOverride is split into multiple classes to avoid having one massive class.
-
     :param appbuilder: The appbuilder.
     """
 
-    """ Models """
+    # Models
     role_model = Role
     permission_model = Permission
     action_model = Action

diff --git a/airflow/auth/managers/fab/security_manager/modules/oauth.py b/airflow/auth/managers/fab/security_manager/modules/oauth.py
@@ -29,11 +29,11 @@
 
 class FabAirflowSecurityManagerOverrideOauth:
     """
+    FabAirflowSecurityManagerOverride is split into multiple classes to avoid having one massive class.
+
     This class contains all methods in
     airflow.auth.managers.fab.security_manager.override.FabAirflowSecurityManagerOverride related to the
     oauth authentication.
-
-    FabAirflowSecurityManagerOverride is split into multiple classes to avoid having one massive class.
     """
 
     def get_oauth_user_info(self, provider, resp):