-
Notifications
You must be signed in to change notification settings - Fork 166
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
feat(promotions): support copying folders and files for Git #2054
Conversation
Signed-off-by: Hidde Beydals <[email protected]>
Signed-off-by: Hidde Beydals <[email protected]>
Signed-off-by: Hidde Beydals <[email protected]>
✅ Deploy Preview for docs-kargo-akuity-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2054 +/- ##
==========================================
+ Coverage 45.84% 45.86% +0.01%
==========================================
Files 236 239 +3
Lines 16121 16332 +211
==========================================
+ Hits 7391 7491 +100
- Misses 8367 8467 +100
- Partials 363 374 +11 ☔ View full report in Codecov by Sentry. |
93913a0
to
68d3cdd
Compare
This moves things under `patches[].copy`, while also adding a `RepoURL` to allow to reference a repository from which things should be copied. Signed-off-by: Hidde Beydals <[email protected]>
Signed-off-by: Hidde Beydals <[email protected]>
Signed-off-by: Hidde Beydals <[email protected]>
68d3cdd
to
783ef6f
Compare
Signed-off-by: Hidde Beydals <[email protected]>
c9ae79b
to
facf756
Compare
I think we can close this since #2536 has this. |
Related to #1250 (and more specifically #1250 (comment)).
This adds minimal support for copying files or directories into user-defined paths during a Git promotion. The primary goal of this pull request is to get a feel of how it could work, as the issue appears to be more of a collection of desired outcomes rather than concrete descriptions of functionalities.