Merge pull request #408 from dephell/jail-no-config

Ignore implicit config in `dephell jail`

dephell/actions/_json.py

@@ -118,7 +118,7 @@ def _beautify(data, *, colors: bool, table: bool) -> str:
         # one dict
         if isinstance(data, dict):
             return tabulate.tabulate(
-                _flatdict(data).items(),
+                sorted(_flatdict(data).items()),
                 headers=('key', 'value'),
                 tablefmt='fancy_grid',
             )

dephell/commands/base.py

@@ -25,6 +25,7 @@
 class BaseCommand(CommandHandler):
     logger = getLogger('dephell.commands')
     prog = 'dephell'
+    find_config = True
 
     @cached_property
     def config(self) -> Config:
@@ -79,6 +80,11 @@ def _attach_config_file(cls, path, env) -> bool:
             config.attach_file(path=path, env=env)
             return True
 
+        # do not implicitly attach file for some commands like `jail`
+        if not cls.find_config:
+            cls.logger.debug('cannot find config file')
+            return False
+
         # if path isn't specified, carefully try default names
         for path in CONFIG_NAMES:
             if not os.path.exists(path):

dephell/commands/generate_authors.py

@@ -11,6 +11,9 @@
 class GenerateAuthorsCommand(BaseCommand):
     """Create AUTHORS file for project by git log.
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/generate_contributing.py

@@ -14,6 +14,8 @@
 class GenerateContributingCommand(BaseCommand):
     """Create CONTRIBUTING.md for DepHell-based project.
     """
+    # because we don't actually use anything from the config
+    find_config = False
     file_name = 'CONTRIBUTING.md'
 
     @staticmethod

dephell/commands/generate_editorconfig.py

@@ -12,6 +12,9 @@ class GenerateEditorconfigCommand(BaseCommand):
     """Create EditorConfig for project.
     https://editorconfig.org/
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/generate_travis.py

@@ -13,9 +13,13 @@
 
 class GenerateTravisCommand(BaseCommand):
     """Create .travis.yml for DepHell-based project.
+
     https://docs.travis-ci.com/user/languages/python/
     https://docs.travis-ci.com/user/customizing-the-build
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/inspect_gadget.py

@@ -47,6 +47,9 @@ class InspectGadgetCommand(BaseCommand):
 
     This command shouldn't be documented.
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/jail_install.py

@@ -16,6 +16,8 @@
 class JailInstallCommand(BaseCommand):
     """Download and install package into isolated environment.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/jail_list.py

@@ -12,6 +12,8 @@
 class JailListCommand(BaseCommand):
     """Show all jails and their entrypoints.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/jail_remove.py

@@ -15,6 +15,8 @@
 class JailRemoveCommand(BaseCommand):
     """Remove package isolated environment.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/jail_show.py

@@ -16,6 +16,8 @@
 class JailShowCommand(BaseCommand):
     """Show info about the package isolated environment.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/jail_try.py

@@ -19,6 +19,8 @@
 class JailTryCommand(BaseCommand):
     """Try packages into temporary isolated environment.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/package_bug.py

@@ -16,6 +16,8 @@
 class PackageBugCommand(BaseCommand):
     """Report bug in a package.
     """
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/package_changelog.py

@@ -17,6 +17,9 @@
 class PackageChangelogCommand(BaseCommand):
     """Find project changelog.
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/self_auth.py

@@ -13,6 +13,8 @@
 class SelfAuthCommand(BaseCommand):
     """Insert, update or delete credentials.
     """
+    # because we don't actually use anything from the config
+    find_config = False
     _global_config_path = get_data_dir() / GLOBAL_CONFIG_NAME
 
     @staticmethod

dephell/commands/self_autocomplete.py

@@ -16,6 +16,8 @@
 class SelfAutocompleteCommand(BaseCommand):
     """Enable DepHell commands autocomplete for current shell.
     """
+    # because we don't actually use anything from the config
+    find_config = False
 
     @staticmethod
     def build_parser(parser) -> ArgumentParser:

dephell/commands/self_uncache.py

@@ -12,6 +12,9 @@
 class SelfUncacheCommand(BaseCommand):
     """Remove dephell cache.
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)

dephell/commands/self_upgrade.py

@@ -14,6 +14,9 @@
 class SelfUpgradeCommand(BaseCommand):
     """Upgrade DepHell to the latest version.
     """
+    # because we don't actually use anything from the config
+    find_config = False
+
     @staticmethod
     def build_parser(parser) -> ArgumentParser:
         builders.build_config(parser)
@@ -27,13 +30,15 @@ def __call__(self) -> bool:
             return self._upgrade_repo(path=path)
         return self._upgrade_package()
 
-    def _upgrade_repo(self, path: Path) -> bool:
+    @staticmethod
+    def _upgrade_repo(path: Path) -> bool:
         with chdir(path):
             cmd = ['git', 'pull', 'origin', 'master']
             result = subprocess.run(cmd)
             return (result.returncode == 0)
 
-    def _upgrade_package(self) -> bool:
+    @staticmethod
+    def _upgrade_package() -> bool:
         manager = PackageManager(executable=Path(sys.executable))
         args = ['-U']
         if manager.is_global:

docs/cmd-jail-show.md

@@ -1,4 +1,4 @@
-# dephell jail list
+# dephell jail show
 
 Shows information about a jail installed by [dephell jail install](cmd-jail-install).