Merge pull request #43 from rails/fix-active-storage

Fix activestorage feature
rails · Aug 8, 2024 · 5c8b176 · 5c8b176
2 parents cf47303 + e1f522c
commit 5c8b176
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/.github/.devcontainer/devcontainer.json b/.github/.devcontainer/devcontainer.json
@@ -12,7 +12,11 @@
       "version": "latest",
       "ppa": "false"
     },
-    "./features/ruby": {}
+    "./features/ruby": {},
+    "./features/activestorage": {},
+    "./features/postgres-client": {},
+    "./features/mysql-client": {},
+    "./features/sqlite3": {}
   },
   // Set `remoteUser` to `root` to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
   "remoteUser": "vscode"

diff --git a/features/activestorage/devcontainer-feature.json b/features/activestorage/devcontainer-feature.json
@@ -1,6 +1,6 @@
 {
     "id": "activestorage",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "name": "Active Storage",
     "description": "Installs utilities needed for Rails Active Storage"
 }
diff --git a/features/activestorage/install.sh b/features/activestorage/install.sh
@@ -4,11 +4,7 @@ set -e
 apt-get update -qq && \
   apt-get install --no-install-recommends -y \
     libvips \
-    #  For video thumbnails
     ffmpeg \
-    # For pdf thumbnails. If you want to use mupdf instead of poppler,
-    # you can install the following packages instead:
-    # mupdf mupdf-tools
     poppler-utils
 
 rm -rf /var/lib/apt/lists/*