A high-throughput and memory-efficient inference and serving engine for LLMs
Mirrored from https://github.com/tduncan-lum/Nvidia-Optimization-Stack---VLLM.git
Synced Nov 15, 11:00 UTC
Fix `dockerfilegraph` pre-commit hook (#17698)
Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>
Harry Mellor committed
May 6, 2025 at 09:56 UTC
05e1f96419281a3a2d58f56b136347b4af1906ba
3 files changed
+6
-5
.pre-commit-config.yaml
-2
@@ -125,8 +125,6 @@ repos:
125
name: Update Dockerfile dependency graph
126
entry: tools/update-dockerfile-graph.sh
127
language: script
128
- files: ^docker/Dockerfile$
129
- pass_filenames: false
128
# Keep `suggestion` last
129
- id: suggestion
130
name: Suggestion
docs/source/assets/contributing/dockerfile-stages-dependency.png
Binary files a/docs/source/assets/contributing/dockerfile-stages-dependency.png and b/docs/source/assets/contributing/dockerfile-stages-dependency.png differ
tools/update-dockerfile-graph.sh
+6
-3
@@ -4,8 +4,11 @@
4
5
set -euo pipefail
6
7
-# Check if docker/Dockerfile is staged for commit
8
-if git diff --cached --name-only | grep -q "^docker/Dockerfile$"; then
7
+# Accept file paths as arguments
8
+FILES=("$@")
9
+
10
+# Check if docker/Dockerfile is among the provided files
11
+if printf '%s\n' "${FILES[@]}" | grep -q "^docker/Dockerfile$"; then
12
echo "docker/Dockerfile has changed, attempting to update dependency graph..."
13
14
# Check if Docker is installed and running
@@ -75,4 +78,4 @@ if git diff --cached --name-only | grep -q "^docker/Dockerfile$"; then
78
fi
79
fi
80
78
-exit 0
81
+exit 0