argoproj-labs / gitops-promoter

A GitOps Environment Promotion Tool
https://argo-gitops-promoter.readthedocs.io
Apache License 2.0
50 stars 5 forks source link

Handle empty status in `Close`, `Update`, and `Merge` #98

Open crenshaw-dev opened 2 weeks ago

crenshaw-dev commented 2 weeks ago

Each of these functions calls prNumber, err := strconv.Atoi(pullRequest.Status.ID). If the status is missing, this fails.

It's especially a problem when deleting a PR that never had the status field populated due to failure to connect to the SCM.