Adressing review feedback

Signed-off-by: Ryan Lettieri <[email protected]>

Author: RyanLettieri

.github/workflows/pr-validation.yml

@@ -37,7 +37,7 @@ jobs:
         pip install -r requirements.txt
     - name: Lint with flake8
       run: |
-        flake8 . --count --show-source --statistics --exit-zero
+        flake8 . --count --show-source --statistics
     - name: Pytest unit tests
       working-directory: tests/durabletask
       run: |

.github/workflows/publish-dts-sdk.yml

@@ -5,12 +5,11 @@ on:
     tags:
       - 'azuremanaged-v*'  # Only run for tags starting with "azuremanaged-v"
 
-# on:
-#   push:
-#     branches: [ "main" ]
-#   pull_request:
-#     branches: [ "main" ]
-
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main" ]
 
 jobs:
   lint:
@@ -31,20 +30,21 @@ jobs:
       working-directory: durabletask-azuremanaged
       run: flake8 .
 
-
   run-docker-tests:
+    env:
+      EMULATOR_VERSION: "v0.0.4"  # Define the variable
     needs: lint
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
 
       - name: Pull Docker image
-        run: docker pull mcr.microsoft.com/dts/dts-emulator:v0.0.4
+        run: docker pull mcr.microsoft.com/dts/dts-emulator:$EMULATOR_VERSION
 
       - name: Run Docker container
         run: |
-          docker run --name dtsemulator -d -p 8080:8080 mcr.microsoft.com/dts/dts-emulator:v0.0.4
+          docker run --name dtsemulator -d -p 8080:8080 mcr.microsoft.com/dts/dts-emulator:$EMULATOR_VERSION
 
       - name: Wait for container to be ready
         run: sleep 10  # Adjust if your service needs more time to start

examples/dts/requirements.txt

@@ -1,8 +1,6 @@
 autopep8
 grpcio>=1.60.0 # 1.60.0 is the version introducing protobuf 1.25.X support, newer versions are backwards compatible
 protobuf
-pytest
-pytest-cov
 azure-identity
 durabletask-azuremanaged
 durabletask