You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm not using it yet as a library, just considering it. aws-nuke would be part of a larger AWS account management service, something that would be responsible for cleaning accounts, where the value of including aws-nuke as a dependency would be to not need to track all of the different AWS services to clean. I could shell out, but there should be more control with context.Context compared to SIGTERM.
If you are going to use it at a library you should look at how the nuke command is implemented and re-implement that function minus the CLI part in my opinion.
You can pass context to the CLI right now too with v2 via os.RunWithContext if I recall correctly.
Either way, v3, is high on my radar, but until it hits it's release I will likely hold on updating it here, or I need better test coverage to ensure things still work properly.
https://cli.urfave.org/migrate-v2-to-v3/
v3
factors outcontext.Context
, in line with modern Go practice.The text was updated successfully, but these errors were encountered: