Remove Circle, fix TTY issue in verify script (#133)

* Remove Circle, fix TTY issue in verify script
demisto · Sep 8, 2024 · 2399589 · 2399589
1 parent 9d817e4
commit 2399589
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 136 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -1,12 +1,7 @@
 name: Build and Release
 
-on:
-  push:
-    branches:
-      - master
-    tags:
-      - 'v*.*.*'
-
+on: pull_request
+
 jobs:
   run-unit-tests:
     strategy:
@@ -44,6 +39,7 @@ jobs:
           path: test-results
 
   verify:
+    if: github.ref == 'refs/heads/master'
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
@@ -116,4 +112,4 @@ jobs:
         env:
           PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }}
         run: |
-          poetry publish -u __token__ -p $PYPI_TOKEN
+          poetry publish -u __token__ -p $PYPI_TOKEN
diff --git a/verify.sh b/verify.sh
@@ -6,7 +6,7 @@ set -e
 # Script to run extra verifications
 
 # Verify that code-gen.sh doesn't generate a diff
-bash -x ./gen-code.sh 
+bash ./gen-code.sh  # Removed the '-x' for verbosity in CI
 # we diff only the code
 DIFF_OUT=$(git diff -- demisto_client)