Update triggers (#2183)

Trying to get this workflow to trigger properly for both main, and for other branches. Currently only trigers automatically for PRs.
nkdAgility · Jul 16, 2024 · 0e1ddc7 · 0e1ddc7
1 parent f6de7a9
commit 0e1ddc7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 23 deletions.
diff --git a/.github/workflows/code-review.yml b/.github/workflows/code-review.yml
@@ -12,10 +12,15 @@ jobs:
   code-review:
     runs-on: ubuntu-latest
     steps:
+      - uses: actions/create-github-app-token@v1
+        id: app-token
+        with:
+          app-id: ${{ env.APP_ID }}
+          private-key: ${{ env.PRIVATE_KEY }}
       - uses: fitomad/github-chatgpt-integration@main
         with:
           openai-api-key: ${{ secrets.OPENAI_API_KEY }}
-          github-token: ${{ secrets.GH_TOKEN }}
+          github-token: ${{ steps.app-token.outputs.token }}
           github-pr-id: ${{ github.event.number }}
           dev-lang: c#
           openai-max-tokens: 4096
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -7,8 +7,9 @@ permissions:
 on: 
   push:
     branches:
-    tags-ignore:
-      - 'v*-*' 
+      - "*"
+    tags:
+      - '!v*-*' 
   pull_request:
     branches:
      - main
@@ -214,25 +215,6 @@ jobs:
           name: AzureDevOpsMigrationTools-Packages
           path: ./staging/**
 
-  # Code Review
-  code-review:
-    name: "Code Review Bot "
-    runs-on: ubuntu-latest
-    needs: [Setup]
-    steps:
-      - uses: actions/create-github-app-token@v1
-        id: app-token
-        with:
-          app-id: ${{ env.APP_ID }}
-          private-key: ${{ env.PRIVATE_KEY }}
-      - uses: fitomad/github-chatgpt-integration@main
-        with:
-          openai-api-key: ${{ secrets.OPENAI_API_KEY }}
-          github-token: ${{ steps.app-token.outputs.token }}
-          github-pr-id: ${{ github.event.number }}
-          dev-lang: c#
-          openai-max-tokens: 4096
-
   # Build Docs
   BuildDocs:
     name: "Build Documentation output"
@@ -381,7 +363,7 @@ jobs:
   # Release to Winget
   WingetRelease:
     name: "Release to Winget"
-    runs-on: ubuntu-latest
+    runs-on: windows-latest
     needs: [Setup, GitHubRelease]
     if: ${{ success() && ( needs.Setup.outputs.nkdAgility_Ring != 'Canary' ) && (needs.Setup.outputs.HasChanged_src) }}
     steps: