Merge pull request #80 from cisagov/add-todo-comments

Reference #79 in a TODO comment
cisagov · Jun 14, 2024 · 0fdc370 · 0fdc370
2 parents ff44677 + 088d33a
commit 0fdc370
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 8 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -185,9 +185,11 @@ jobs:
           - debian10-systemd
           - debian11-systemd
           - debian12-systemd
-          # Docker does not yet officially support Debian Trixie.  See
-          # https://docs.docker.com/engine/install/debian/ for more
-          # details.
+          # TODO: Docker does not yet officially support Debian
+          # Trixie, but we should add support when that becomes
+          # possible.  See
+          # https://docs.docker.com/engine/install/debian/ and
+          # cisagov/ansible-role-docker#79 for more details.
           # - debian13-systemd
           - fedora39-systemd
           - fedora40-systemd

diff --git a/meta/main.yml b/meta/main.yml
@@ -30,9 +30,10 @@ galaxy_info:
         - buster
         - bullseye
         - bookworm
-        # Docker does not yet officially support Debian Trixie.  See
-        # https://docs.docker.com/engine/install/debian/ for more
-        # details.
+        # TODO: Docker does not yet officially support Debian Trixie,
+        # but we should add support when that becomes possible.  See
+        # https://docs.docker.com/engine/install/debian/ and
+        # cisagov/ansible-role-docker#79 for more details.
         # - trixie
     - name: Fedora
       versions:

diff --git a/molecule/default/molecule.yml b/molecule/default/molecule.yml
@@ -76,8 +76,10 @@ platforms:
     privileged: true
     volumes:
       - /sys/fs/cgroup:/sys/fs/cgroup:rw
-  # Docker does not yet officially support Debian Trixie.  See
-  # https://docs.docker.com/engine/install/debian/ for more details.
+  # TODO: Docker does not yet officially support Debian Trixie, but we
+  # should add support when that becomes possible.  See
+  # https://docs.docker.com/engine/install/debian/ and
+  # cisagov/ansible-role-docker#79 for more details.
   # - cgroupns_mode: host
   #   command: /lib/systemd/systemd
   #   image: docker.io/cisagov/docker-debian13-ansible:latest