Merge pull request #56 from rails/simplify-bundler-cache

Simplify bundler cache feature
rails · Oct 4, 2024 · 54c5e33 · 54c5e33
2 parents e0a7118 + 073e4ba
commit 54c5e33
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 18 deletions.
diff --git a/features/bundler-cache/devcontainer-feature.json b/features/bundler-cache/devcontainer-feature.json
@@ -1,6 +1,6 @@
 {
   "id": "bundler-cache",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "name": "Bundler cache",
   "description": "Creates a volume for persisting the installed gems across different containers",
   "containerEnv": {
@@ -13,5 +13,5 @@
       "type": "volume"
     }
   ],
-  "postCreateCommand": "/usr/local/share/bundler-data-permissions.sh"
+  "postCreateCommand": "sudo chown -R ${USER} /bundle"
 }
diff --git a/features/bundler-cache/install.sh b/features/bundler-cache/install.sh
@@ -1,16 +1 @@
-#!/bin/sh
-
-set -e
-
-USERNAME="${USERNAME:-"${_REMOTE_USER:-"automatic"}"}"
-
-POST_CREATE_COMMAND_SCRIPT_PATH="/usr/local/share/bundler-data-permissions.sh"
-
-tee "$POST_CREATE_COMMAND_SCRIPT_PATH" > /dev/null \
-<< EOF
-#!/bin/sh
-set -e
-sudo chown -R ${USERNAME} /bundle
-EOF
-
-chmod 755 "$POST_CREATE_COMMAND_SCRIPT_PATH"
+# This is a no op