From 6171d82cee880ed8434f4c829da53c926d34a384 Mon Sep 17 00:00:00 2001 From: chipmunkie Date: Mon, 8 Jan 2024 11:07:36 -0500 Subject: [PATCH] Fix procedure for python dependency updates (#1059) Our current python dependency update command doesn't update the requirements.txt file when the file has contents. Removing the contents allows for the it to be written with updated dependencies. Signed-off-by: chipmunkie --- docs/root/updating_envoy_dependency.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/docs/root/updating_envoy_dependency.md b/docs/root/updating_envoy_dependency.md index 92bf22d45..72046e9ca 100644 --- a/docs/root/updating_envoy_dependency.md +++ b/docs/root/updating_envoy_dependency.md @@ -293,10 +293,12 @@ These should always be accompanied with a comment explaining them. Avoid using First attempt to remove all existing pins in [tools/base/requirements.in](/tools/base/requirements.in) to see if they are still necessary. Once done editing -[tools/base/requirements.in](/tools/base/requirements.in), update the +[tools/base/requirements.in](/tools/base/requirements.in), delete the contents +of [tools/base/requirements.txt](/tools/base/requirements.txt) and update the dependencies by running: ```bash +echo > tools/base/requirements.txt bazel run //tools/base:requirements.update ```