Fix: Resolve several issues with the require dependencies decorator (#315)

Fix several issues re. the requires_dependencies decorator:
* There was a missing space between the sentences.
* Crucial brackets were missing in making the error message.
* "pygithub" was used where "github" should have been used.

Author: tomaarsen

CHANGELOG.md

@@ -1,9 +1,20 @@
+## 0.5.1-dev1
+
+### Enhancements
+
+### Features
+
+### Fixes
+
+* Fix several issues with the `requires_dependencies` decorator, including the error message
+  and how it was used.
+
 ## 0.5.0
 
 ### Enhancements
 
 * Add `requires_dependencies` Python decorator to check dependencies are installed before
-instantiating a class or running a function
+  instantiating a class or running a function
 
 ### Features
 

unstructured/__version__.py

@@ -1 +1 @@
-__version__ = "0.5.0"  # pragma: no cover
+__version__ = "0.5.1-dev1"  # pragma: no cover

unstructured/ingest/connector/github.py

@@ -125,7 +125,7 @@ def write_result(self):
         print(f"Wrote {output_filename}")
 
 
-@requires_dependencies(["pygithub"], extras="github")
+@requires_dependencies(["github"], extras="github")
 class GitHubConnector(BaseConnector):
     def __init__(self, config: SimpleGitHubConfig):
         from github import Github

unstructured/utils.py

@@ -14,7 +14,10 @@ def read_from_jsonl(filename: str) -> List[Dict]:
         return [json.loads(line) for line in input_file]
 
 
-def requires_dependencies(dependencies: Union[str, List[str]], extras: Optional[str] = None):
+def requires_dependencies(
+    dependencies: Union[str, List[str]],
+    extras: Optional[str] = None,
+):
     if isinstance(dependencies, str):
         dependencies = [dependencies]
 
@@ -29,10 +32,12 @@ def wrapper(*args, **kwargs):
                     missing_deps.append(dep)
             if len(missing_deps) > 0:
                 raise ImportError(
-                    f"Following dependencies are missing: {', '.join(missing_deps)}."
-                    + f"Please install them using `pip install unstructured[{extras}]`."
-                    if extras
-                    else f"Please install them using `pip install {' '.join(missing_deps)}`.",
+                    f"Following dependencies are missing: {', '.join(missing_deps)}. "
+                    + (
+                        f"Please install them using `pip install unstructured[{extras}]`."
+                        if extras
+                        else f"Please install them using `pip install {' '.join(missing_deps)}`."
+                    ),
                 )
             return func(*args, **kwargs)