addressed comments

Author: Divyansh-db

databricks/sdk/credentials_provider.py

@@ -435,7 +435,9 @@ def github_oidc(cfg: "Config") -> Optional[CredentialsProvider]:
     Supported in GitHub Actions with OIDC service connections.
     """
     return _oidc_credentials_provider(
-        cfg=cfg, supplier_factory=lambda: oidc_token_supplier.GitHubOIDCTokenSupplier(), provider_name="GitHub OIDC"
+        cfg=cfg,
+        supplier_factory=lambda: oidc_token_supplier.GitHubOIDCTokenSupplier(),
+        provider_name="GitHub OIDC",
     )
 
 

databricks/sdk/oidc_token_supplier.py

@@ -7,6 +7,7 @@
 logger = logging.getLogger("databricks.sdk")
 
 
+# TODO: Check the required environment variables while creating the instance rather than in the get_oidc_token method to allow early return.
 class GitHubOIDCTokenSupplier:
     """
     Supplies OIDC tokens from GitHub Actions.