Cleanup PR

pyscript · JeffersGlass · May 24, 2023 · Apr 1, 2023 · Apr 1, 2023 · Apr 1, 2023
commit 0443dfcfc32e03391b6c6fa50a9fd4049b352b07
diff --git a/pyscriptjs/src/main.ts b/pyscriptjs/src/main.ts
@@ -23,9 +23,6 @@ import * as Synclink from 'synclink';
 
 const logger = getLogger('pyscript/main');
 
-// This comment is to force CI to run again on this branch, and will be removed
-// before final review
-
 /**
  * Monkey patching the error transfer handler to preserve the `$$isUserError`
  * marker so as to detect `UserError` subclasses in the error handling code.

diff --git a/pyscriptjs/src/remote_interpreter.ts b/pyscriptjs/src/remote_interpreter.ts
@@ -170,7 +170,7 @@ export class RemoteInterpreter extends Object {
      * */
     async loadPackage(names: string | string[]): Promise<void> {
         logger.info(`pyodide.loadPackage: ${names.toString()}`);
-        // the new way in Pyodode 0.22.0 and later a locally downloaded older version of pyodide
+        // the new way in Pyodode 0.22 and later a locally downloaded older version of pyodide
         // for which the signature of `loadPackage` accepts the above params as args i.e.
         // the call uses `logger.info.bind(logger), logger.info.bind(logger)`.
         const messageCallback = logger.info.bind(logger) as typeof logger.info;

diff --git a/pyscriptjs/tests/integration/test_py_config.py b/pyscriptjs/tests/integration/test_py_config.py
@@ -82,7 +82,7 @@ def test_py_config_external(self):
 
     # The default pyodide version is newer than
     # the one we are loading below (after downloading locally)
-    # which is 0.20.0
+    # which is 0.22.0
 
     # The test checks if loading a different interpreter is possible
     # and that too from a locally downloaded file without needing

diff --git a/pyscriptjs/tests/integration/test_zzz_docs_snippets.py b/pyscriptjs/tests/integration/test_zzz_docs_snippets.py
@@ -130,6 +130,7 @@ def test_tutorials_py_config_fetch(self):
         self.assert_no_banners()
 
     def test_tutorials_py_config_interpreter(self):
+        """Load a previous version of Pyodide"""
         self.pyscript_run(
             """
             <py-config>