-
Notifications
You must be signed in to change notification settings - Fork 360
Read test commands flags from ./config.yaml file #385
Comments
Do we have to just paste this code |
Hey @noaabarki I’d be interested in taking this on |
Hi @laneia! That's amazing, you got it :) |
Will this cause us to override the flags without the ability to change it via the cli? To be more specific, I will be able to manually add |
As with most tools, I think we want the inline flags to merge & override the flags from config.yaml |
@noaabarki I would like to work on this issue :) |
You are the best! 🥳 It's yours. |
Hey @noaabarki, Wanted to get it done within the day that I was assigned but got a bit stuck, |
@noaabarki is anyone working on this issue? |
@amustaque97 Nope, feel free to give it a shot :) |
@noaabarki is there anyone working on this issue? |
@Abhi-y2003 go for it! |
Describe the solution you'd like
Read all test command’s flags from both command and the config file(
~/.datree/config.yaml
). The main feature is to allow users to set permanent yet custom flags in their config local settings.“How to implementation” Suggestion
Make sure all flags and options are in
TestCommandFlags
(wrapTestCommandFlags
inTestCommandOptions
as you see best).Change
localConfig.GetLocalConfiguration()
to return all the options from./config.yaml
file.Use
GetLocalConfiguration()
outside the scope of Cobra’s commandRunE
and initiate flags with local config. Eventually, the outcome should be something likeRequirements
Golang basic level.
The text was updated successfully, but these errors were encountered: