]> granicus.if.org Git - python/commitdiff
Swap 'if' branches so content matches to condition in importlib example (GH-14947)
authorTzu-ping Chung <uranusjr@gmail.com>
Thu, 25 Jul 2019 17:20:33 +0000 (01:20 +0800)
committerMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Thu, 25 Jul 2019 17:20:33 +0000 (10:20 -0700)
Prior to this change the guard on an 'elif' used an assignment expression whose value was used in a later 'else' block, causing some confusion for people.

(Discussion on Twitter: https://twitter.com/brettsky/status/1153861041068994566.)

Automerge-Triggered-By: @brettcannon
Doc/library/importlib.rst

index 9fab0779aa74211cab77339a061380bf9a1d3022..65a685022e92f1886dbe7e62276a421dcf5ecaef 100644 (file)
@@ -1640,14 +1640,14 @@ import, then you should use :func:`importlib.util.find_spec`.
 
   if name in sys.modules:
       print(f"{name!r} already in sys.modules")
-  elif (spec := importlib.util.find_spec(name)) is None:
-      print(f"can't find the {name!r} module")
-  else:
+  elif (spec := importlib.util.find_spec(name)) is not None:
       # If you chose to perform the actual import ...
       module = importlib.util.module_from_spec(spec)
       sys.modules[name] = module
       spec.loader.exec_module(module)
       print(f"{name!r} has been imported")
+  else:
+      print(f"can't find the {name!r} module")
 
 
 Importing a source file directly