[PR #457] [CLOSED] JUnit report and exit code issues #541

New issue

Closed

opened 2026-02-26 22:31:26 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 22:31:26 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/457
Author: @cursor[bot]
Created: 2/9/2026
Status: ❌ Closed

Base: ci-exit-codes ← Head: cursor/junit-report-and-exit-code-issues-69a0

📝 Commits (2)

8af59f7 feat: add CI/CD exit codes and JUnit reporting
de8729c fix: remove double XML escaping, correct exit code constants, remove unused WriteTo

📊 Changes

12 files changed (+741 additions, -27 deletions)

View changed files

➕ cmd/exit_codes.go (+105 -0)
➕ cmd/exit_codes_test.go (+99 -0)
📝 cmd/run.go (+60 -13)
📝 internal/asc/client_http.go (+14 -8)
📝 internal/asc/errors.go (+7 -3)
📝 internal/asc/notary.go (+1 -1)
📝 internal/cli/cmdtest/error_hints_test.go (+2 -2)
➕ internal/cli/cmdtest/exit_codes_test.go (+88 -0)
➕ internal/cli/shared/ci_flags.go (+40 -0)
➕ internal/cli/shared/junit_report.go (+155 -0)
➕ internal/cli/shared/junit_report_test.go (+169 -0)
📝 internal/cli/shared/shared.go (+1 -0)

📄 Description

Address three bugs: correct JUnit report XML escaping, update HTTP exit code constants, and remove an unused WriteTo method.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/457 **Author:** [@cursor[bot]](https://github.com/apps/cursor) **Created:** 2/9/2026 **Status:** ❌ Closed **Base:** `ci-exit-codes` ← **Head:** `cursor/junit-report-and-exit-code-issues-69a0` --- ### 📝 Commits (2) - [`8af59f7`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/8af59f7ba233bce40435b4f87ba94f4a132fcc36) feat: add CI/CD exit codes and JUnit reporting - [`de8729c`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/de8729c812c6d1266c6ceef51cdf287220ed36ae) fix: remove double XML escaping, correct exit code constants, remove unused WriteTo ### 📊 Changes **12 files changed** (+741 additions, -27 deletions) <details> <summary>View changed files</summary> ➕ `cmd/exit_codes.go` (+105 -0) ➕ `cmd/exit_codes_test.go` (+99 -0) 📝 `cmd/run.go` (+60 -13) 📝 `internal/asc/client_http.go` (+14 -8) 📝 `internal/asc/errors.go` (+7 -3) 📝 `internal/asc/notary.go` (+1 -1) 📝 `internal/cli/cmdtest/error_hints_test.go` (+2 -2) ➕ `internal/cli/cmdtest/exit_codes_test.go` (+88 -0) ➕ `internal/cli/shared/ci_flags.go` (+40 -0) ➕ `internal/cli/shared/junit_report.go` (+155 -0) ➕ `internal/cli/shared/junit_report_test.go` (+169 -0) 📝 `internal/cli/shared/shared.go` (+1 -0) </details> ### 📄 Description Address three bugs: correct JUnit report XML escaping, update HTTP exit code constants, and remove an unused `WriteTo` method. --- --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>