Fix except‑pass warnings, enable Jinja2 autoescape, clean docs (tab →…

.github/workflows/github-actions.yml

@@ -64,6 +64,8 @@ jobs:
       if: runner.os == 'Windows'
       run: git config --global core.longpaths true
     - name: test
+      env:
+        CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
       run: |
         mkdir -p ~/.ssh
         tox -e ${{ matrix.tox_env }} -v

docs/conf.py

@@ -22,11 +22,11 @@
 
 source_suffix = '.rst'
 master_doc = 'index'
-project = u'travis2docker'
+project = 'travis2docker'
 year = '2016'
-author = u'Vauxoo'
+author = 'Vauxoo'
 copyright = '{0}, {1}'.format(year, author)
-version = release = u'7.0.5'
+version = release = '7.0.5'
 
 pygments_style = 'trac'
 templates_path = ['.']

docs/usage.rst

@@ -4,4 +4,4 @@ Usage
 
 To use travis2docker in a project::
 
-	import travis2docker
+    import travis2docker

src/travis2docker/git_run.py

@@ -1,4 +1,4 @@
-# pylint: disable=useless-object-inheritance,except-pass
+# ruff: disable[useless-object-inheritance, try-except-pass]
 
 import contextlib
 import logging

src/travis2docker/travis2docker.py

@@ -1,4 +1,5 @@
-# pylint: disable=useless-object-inheritance,consider-using-with
+# pylint: disable=consider-using-with
+# ruff: disable[useless-object-inheritance]
 import json
 import logging
 import pathlib
@@ -54,8 +55,8 @@ def get_vscode_version():
                 lines = subprocess.check_output([code_bin, "--version"]).decode("UTF-8").splitlines()
                 if len(lines) >= 2 and lines[1].strip():
                     return "commit:%s" % lines[1].strip()
-            except (subprocess.CalledProcessError, OSError):
-                pass
+            except (subprocess.CalledProcessError, OSError) as e:
+                _logger.debug("Failed to get VS Code version: %s", e)
         return "latest"
 
     def __init__(
@@ -104,7 +105,10 @@ def __init__(
             templates_path = templates_dir
         self.copy_paths = copy_paths
         self.os_kwargs = os_kwargs
-        self.jinja_env = jinja2.Environment(loader=jinja2.FileSystemLoader(templates_path))
+        self.jinja_env = jinja2.Environment(
+            loader=jinja2.FileSystemLoader(templates_path),
+            autoescape=jinja2.select_autoescape(['html', 'xml', 'json'])
+        )
         self.image = image
         if work_path is None:
             self.work_path = pathlib.Path(gettempdir()) / pathlib.Path(__file__).stem
@@ -155,7 +159,7 @@ def compute_devcontainer(self):
         """Generate a .devcontainer.json so VS Code offers reopening the generated
         image as a Development Container"""
         name = self.os_kwargs.get("repo_project") or self.variables_sh_data.get("main_app", "odoo")
-        devcontainer_content = self.jinja_env.get_template("devcontainer.json").render(
+        devcontainer_content = self.jinja_env.get_template("devcontainer.json.jinja").render(
             image=self.new_image, name=name, **self.os_kwargs
         )
         (self.curr_work_path / ".devcontainer.json").write_text(devcontainer_content)
@@ -169,8 +173,8 @@ def copy_path(self, path):
         if src.is_dir():
             try:
                 shutil.copytree(src, dest_path)
-            except shutil.Error:  # ruff: ignore[except-pass]
-                pass  # There are permissions errors to copy
+            except shutil.Error as e:
+                _logger.debug("Failed to copy directory %s to %s: %s", src, dest_path, e)
         elif src.is_file():
             shutil.copy(src, dest_path)
         else: