-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove unnecessary push when rendering to local path #272
Conversation
Signed-off-by: Kent Rancourt <[email protected]>
✅ Deploy Preview for docs-kargo-render-akuity-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
if localTargetBranchExists { | ||
logger.Debug("target branch exists locally") | ||
if err = rc.repo.Checkout(rc.request.TargetBranch); err != nil { | ||
return fmt.Errorf("error checking out target branch: %w", err) | ||
} | ||
logger.Debug("checked out target branch") | ||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added for good measure; instead of blindly creating an orphan branch.
if rc.request.LocalOutPath != "" { | ||
return nil // There's no need to push the new branch to the remote | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the important bit.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #272 +/- ##
==========================================
+ Coverage 27.72% 27.75% +0.03%
==========================================
Files 22 22
Lines 2103 2133 +30
==========================================
+ Hits 583 592 +9
- Misses 1441 1461 +20
- Partials 79 80 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on my current (new-found) understanding of how this (should) work(s). This appears to address the issue.
No description provided.