Merge branch 'main' into feature/add-anonymize-option

k8sgpt-ai · Apr 11, 2023 · fe502e1 · fe502e1
2 parents 705d2a0 + 21dc61c
commit fe502e1
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -9,4 +9,4 @@
 # Unless a later match takes precedence, these owners will be requested for
 # review when someone opens a pull request.
 
-* @k8sgpt-ai/maintainers
+* @k8sgpt-ai/maintainers @k8sgpt-ai/k8sgpt-approvers
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
@@ -25,7 +25,7 @@ jobs:
       - name: Checkout
         uses: actions/checkout@8f4b7f84864484a7bf31766abe9204da3cbe65b3 # v3
 
-      - uses: google-github-actions/release-please-action@ee9822ec2c397e8a364d634464339ac43a06e042 # v3
+      - uses: google-github-actions/release-please-action@f7edb9e61420a022c0f1123edaf342d7e127df92 # v3
         id: release
         with:
           command: manifest

diff --git a/pkg/analysis/analysis.go b/pkg/analysis/analysis.go
@@ -153,7 +153,11 @@ func (a *Analysis) GetAIResults(output string, anonymize bool) error {
 		}
 		parsedText, err := a.AIClient.Parse(a.Context, texts, a.NoCache)
 		if err != nil {
-			bar.Exit()
+			// FIXME: can we avoid checking if output is json multiple times?
+			//   maybe implement the progress bar better?
+			if output != "json" {
+				bar.Exit()
+			}
 
 			// Check for exhaustion
 			if strings.Contains(err.Error(), "status code: 429") {