Prefer checking user over touching a file

Parnassius · Sep 10, 2021 · 8aae45a · 8aae45a
1 parent 6d04a3c
commit 8aae45a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Dockerfile.development b/Dockerfile.development
@@ -2,7 +2,6 @@ FROM dependabot/dependabot-core
 
 # Temporarily switch to root user in order to install packages
 USER root
-RUN touch /etc/dependabot-core-dev
 RUN apt-get update \
   && apt-get install -y vim strace ltrace gdb shellcheck \
   && rm -rf /var/lib/apt/lists/*

diff --git a/bin/dry-run.rb b/bin/dry-run.rb
@@ -35,7 +35,8 @@
 
 # rubocop:disable Style/GlobalVars
 
-unless File.exist?("/etc/dependabot-core-dev")
+require "etc"
+unless Etc.getpwuid(Process.uid).name == "dependabot"
   puts <<~INFO
     bin/dry-run.rb is only supported in a developerment container.