Merge pull request #6 from uche-madu/dev

fix workflow env variable name
uche-madu · Sep 30, 2023 · 7672e5a · 7672e5a
2 parents e293e97 + 0f9865e
commit 7672e5a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/retag.yaml b/.github/workflows/retag.yaml
@@ -61,7 +61,7 @@ jobs:
 
       - name: Retag Docker Image
         run: |
-            echo "SHORT_SHA=${GITHUB_SHA::8}" >> $GITHUB_ENV
+            SHORT_SHA=${GITHUB_SHA::8}
             docker pull "$GAR_LOCATION-docker.pkg.dev/$PROJECT_ID/$REPOSITORY/$IMAGE:$FEATURE_BRANCH_NAME"
             docker tag "$GAR_LOCATION-docker.pkg.dev/$PROJECT_ID/$REPOSITORY/$IMAGE:$FEATURE_BRANCH_NAME" "$GAR_LOCATION-docker.pkg.dev/$PROJECT_ID/$REPOSITORY/$IMAGE:$SHORT_SHA"
             docker push "$GAR_LOCATION-docker.pkg.dev/$PROJECT_ID/$REPOSITORY/$IMAGE:$SHORT_SHA"

diff --git a/Dockerfile b/Dockerfile
@@ -9,7 +9,7 @@ WORKDIR /usr/local/airflow
 COPY requirements.txt .
 
 # Install dbt into a virtual environment for use in ExternalPythonOperator
-# The sed command allows pip to avoid running with --user flag in virtual environment
+# The sed command allows pip to avoid running with --user flag in the virtual environment
 RUN python -m venv dbt_venv && \
     sed -i 's/include-system-site-packages = false/include-system-site-packages = true/' dbt_venv/pyvenv.cfg && \
     . dbt_venv/bin/activate && \