Merge pull request #36 from OpenRL-Lab/dev

Dev

Author: huangshiyu13

openplugin/cli/cli.py

@@ -15,7 +15,7 @@
 # limitations under the License.
 
 """"""
-import os
+
 
 import click
 from click.core import Context, Option

openplugin/install/local_install.py

@@ -21,13 +21,13 @@
 import tempfile
 import zipfile
 
+from openplugin.utils.check import check_local_plugin
 from openplugin.utils.local_plugin_utils import get_local_plugin_name
 from openplugin.utils.util import (
     get_plugin_directory,
     get_plugin_version,
     make_zip_file,
 )
-from openplugin.utils.check import check_local_plugin
 
 
 def install_local_plugin() -> bool:
@@ -41,7 +41,9 @@ def install_local_plugin() -> bool:
         return False
     tempdir = tempfile.mkdtemp()
     filepath = make_zip_file(
-        dir_to_put_file_in=tempdir, plugin_directory=local_plugin_directory, plugin_name=plugin_name
+        dir_to_put_file_in=tempdir,
+        plugin_directory=local_plugin_directory,
+        plugin_name=plugin_name,
     )
 
     z = zipfile.ZipFile(filepath)

openplugin/template/base_template.py

@@ -17,9 +17,7 @@
 """"""
 from abc import ABC, abstractmethod
 from pathlib import Path
-from typing import Any, Dict, List, Optional, Tuple, Union
-
-from openplugin.template.utils import render_ROOT_URL
+from typing import Union
 
 
 class BaseTemplate(ABC):

openplugin/utils/check.py

@@ -15,12 +15,12 @@
 # limitations under the License.
 
 """"""
-from typing import Tuple, Union
 import json
-
 from pathlib import Path
+from typing import Tuple, Union
+
 
-def check_info_file(info_file:Union[str,Path]) -> Tuple[bool, str]:
+def check_info_file(info_file: Union[str, Path]) -> Tuple[bool, str]:
     """Check if the given info file is valid."""
     message = ""
     success = True
@@ -57,5 +57,5 @@ def check_local_plugin(plugin_path: Union[str, Path]) -> Tuple[bool, str]:
         success = False
         message = f"Plugin path {plugin_path} does not contain all required files."
     else:
-        success, message = check_info_file(plugin_path/"info.json")
+        success, message = check_info_file(plugin_path / "info.json")
     return success, message