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(core): set can handle all positional args together #3722

Merged

Conversation

Mia-Cross
Copy link
Contributor

This PR introduces a new behavior for the set verb when there are several positional arguments.

This feature will be helpful for #3597 but for now, there is no set command that is using positional arguments, so the test in this PR only shows that the already existing commands with set still work. More tests will be added in #3597.

@Mia-Cross Mia-Cross self-assigned this Mar 21, 2024
@codecov-commenter
Copy link

codecov-commenter commented Mar 21, 2024

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 77.12%. Comparing base (5bf53e0) to head (69e38be).
Report is 11 commits behind head on master.

Files Patch % Lines
internal/core/cobra_utils.go 84.61% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3722      +/-   ##
==========================================
+ Coverage   75.45%   77.12%   +1.67%     
==========================================
  Files         202      202              
  Lines       44323    37142    -7181     
==========================================
- Hits        33444    28646    -4798     
+ Misses       9653     7267    -2386     
- Partials     1226     1229       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Mia-Cross Mia-Cross force-pushed the core_set_with_all_positional_args branch from 4dc793d to 4fd35a5 Compare March 28, 2024 17:11
@Mia-Cross Mia-Cross marked this pull request as draft March 29, 2024 10:22
@Mia-Cross Mia-Cross force-pushed the core_set_with_all_positional_args branch 4 times, most recently from 7f5c3d8 to dc6c4c0 Compare April 2, 2024 12:33
@Mia-Cross Mia-Cross marked this pull request as ready for review April 2, 2024 13:33
@Mia-Cross Mia-Cross force-pushed the core_set_with_all_positional_args branch from 0bf80a5 to 8067653 Compare April 8, 2024 13:29
internal/core/cobra_utils_test.go Show resolved Hide resolved
internal/core/command.go Outdated Show resolved Hide resolved
Mia-Cross and others added 2 commits April 9, 2024 11:47
Co-authored-by: Jules Castéran <jules.casteran@gmail.com>
@Codelax Codelax added this pull request to the merge queue Apr 9, 2024
Merged via the queue into scaleway:master with commit d54c911 Apr 9, 2024
12 checks passed
@Mia-Cross Mia-Cross deleted the core_set_with_all_positional_args branch April 9, 2024 14:37
jremy42 pushed a commit to jremy42/scaleway-cli that referenced this pull request Apr 12, 2024
Co-authored-by: Jules Castéran <jules.casteran@gmail.com>
Laure-di pushed a commit to Laure-di/scaleway-cli that referenced this pull request Apr 16, 2024
Co-authored-by: Jules Castéran <jules.casteran@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants