feat: Add warning for missing upstream during 'keptn create project' #6433
Conversation
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
7ef6d14
to
d1f1815
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Codecov Report
@@ Coverage Diff @@
## master #6433 +/- ##
==========================================
- Coverage 57.65% 54.28% -3.38%
==========================================
Files 466 336 -130
Lines 26560 22613 -3947
Branches 1810 1167 -643
==========================================
- Hits 15314 12275 -3039
+ Misses 10219 9376 -843
+ Partials 1027 962 -65
|
@@ -25,7 +26,7 @@ type createProjectCmdParams struct { | |||
var createProjectParams *createProjectCmdParams | |||
|
|||
const gitErrMsg = `Please specify a 'git-user', 'git-token', and 'git-remote-url' as flags for configuring a Git upstream repository` | |||
const gitMissingUpstream = `WARNING: Creating a project without Git upstream repository is not recommended. | |||
const gitMissingUpstream = `WARNING: Creating a project without Git upstream repository is not recommended and will not be supported in the future anymore. |
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.
I think we should be a bit more specific here, e.g., stating which version we plan to deprecate this support.
Though you have implemented it according to #6306, so I'll approve this.
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.
I can add a specific version there, it is not a problem, but do we already know in which version we want to deprecate the support ?
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.
LGTM (though please see my comment)
Signed-off-by: odubajDT ondrej.dubaj@dynatrace.com
This PR
Related Issues
Fixes #6306