Apply suggestions from code review

Co-authored-by: Martin Hjelmare <marhje52@gmail.com>

homeassistant/components/blueprint/importer.py

@@ -90,7 +90,7 @@ def _get_community_post_import_url(url: str) -> str:
 def _extract_blueprint_from_community_topic(
     url,
     topic,
-) -> ImportedBlueprint:
+) -> Optional[ImportedBlueprint]:
     """Extract a blueprint from a community post JSON.
 
     Async friendly.
@@ -167,9 +167,9 @@ async def fetch_blueprint_from_github_url(
 
 async def fetch_blueprint_from_url(hass: HomeAssistant, url: str) -> ImportedBlueprint:
     """Get a blueprint from a url."""
-    for meth in (fetch_blueprint_from_community_post, fetch_blueprint_from_github_url):
+    for func in (fetch_blueprint_from_community_post, fetch_blueprint_from_github_url):
         try:
-            return await meth(hass, url)
+            return await func(hass, url)
         except ValueError:
             pass
 

homeassistant/components/blueprint/models.py

@@ -191,7 +191,7 @@ async def async_get_blueprints(
         async with self._load_lock:
             return await self.hass.async_add_executor_job(self._load_blueprints)
 
-    async def async_get_blueprint(self, blueprint_name: str) -> Blueprint:
+    async def async_get_blueprint(self, blueprint_path: str) -> Blueprint:
         """Get a blueprint."""
         if blueprint_name in self._blueprints:
             return self._blueprints[blueprint_name]

homeassistant/components/blueprint/schemas.py

@@ -37,7 +37,7 @@ def is_blueprint_instance_config(config: Any) -> bool:
 )
 
 
-def validate_yaml_suffix(value):
+def validate_yaml_suffix(value: str) -> str:
     """Validate value has a YAML suffix."""
     if not value.endswith(".yaml"):
         raise vol.Invalid("Path needs to end in .yaml")