add WithContext to set context.context

[crochee]

No description provided.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

crochee seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[marckhouzam]

Thanks @crochee.
To set the context you can use ExecuteContext(..) or ExecuteContextC(..).
Would that be sufficient?

[crochee]

ExecuteContext or ExecuteContextC executes the contents of the whole process, but I want to set it before executing the run of the whole process, because some flags at this stage already have values, and I will set its values in the context.for example:
func setInfo(cmd *cobra.Command, _ []string) error {
ctx := v2.SetAccountID(cmd.Context(), viper.GetString("account"))
cmd.WithContext(ctx)
return nil
}

[joshcarp]

Would really like this feature;
I want to use a functional option pattern and this is an example that I currently have in my code

type GrptlOption func(*cobra.Command) error

func WithFileDescriptors(descriptors ...protoreflect.FileDescriptor) GrptlOption {
	return func(cmd *cobra.Command) error {
		err := CommandFromFileDescriptors(cmd, descriptors...)
		if err != nil {
			return err
		}
		return nil
	}
}

I've got a lot more options and this pattern allows me to have nice abstracted pieces of logic that interact with the cobra command.

The issue I'm facing at the moment is that I can't write a nifty option (something like WithAuthentication) because that needs access to reading and setting the context. Currently I can only read the context.

Needing to use ExecuteContext breaks this abstraction because I need to run it right at the end