Skip to content
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(init): no overwrite prompt when adding a new profile #3097

Merged

Conversation

Codelax
Copy link
Member

@Codelax Codelax commented Apr 27, 2023

Improvement for init rework #2792
Lookup #2792 (comment)

Now we get an override prompt only when doing init for an existing profile. There is not prompt for scw init -p mynewprofile
Also the override prompt has changed, it only print the profile that will be override instead of the whole config

@Codelax Codelax force-pushed the feat/init-no-overwrite-for-new-profiles branch from 9dc1302 to aa56c5a Compare April 27, 2023 09:20
@codecov-commenter
Copy link

Codecov Report

Merging #3097 (aa56c5a) into master (68609e8) will increase coverage by 0.00%.
The diff coverage is 82.35%.

@@           Coverage Diff           @@
##           master    #3097   +/-   ##
=======================================
  Coverage   75.79%   75.80%           
=======================================
  Files         169      169           
  Lines       36631    36638    +7     
=======================================
+ Hits        27765    27772    +7     
  Misses       7903     7903           
  Partials      963      963           
Impacted Files Coverage Δ
internal/namespaces/init/init.go 68.57% <40.00%> (+0.15%) ⬆️
internal/namespaces/init/prompt.go 24.11% <100.00%> (+3.37%) ⬆️

@Codelax Codelax marked this pull request as ready for review April 27, 2023 09:42
@remyleone remyleone added this pull request to the merge queue Apr 27, 2023
Merged via the queue into scaleway:master with commit 4aa9160 Apr 27, 2023
11 checks passed
@Codelax Codelax mentioned this pull request May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants