Denis Vaumoron
Denis Vaumoron
Hi @Adesoji1, I dont get your 4, using a wrapper calling an other program doesn't seem to be the goal here, a direct access with embedding to avoid a separate...
After a quick check into the code, writing a wrapper to internal part need at most to start by changing the meta.View type to an interface (to allow to retrieve...
To be clearer, changing the meta.View will allow the library to return a tfdiags.Diagnostics to the caller (this type and others need to be moved outside of the internal part...
I worked recently on https://github.com/dvaumoron/gotofuenv, maybe using some part of it as a library could help to install tofu before launching it ?
After a look at tofu-exec, it seems that it already allows to install a tofu version before launching it...
I take a look at ConsolidateWarnings, as the comment indicate, this method aggregate warnings with the same summary (which is "Argument is deprecated" in the example), the -1 special case...
DiagnosticWarningsCompact can handle consolidated warnings, so conflicting when the two optional flags are present is not mandatory (even if it seems strange to use them at the same time). The...
The view part of cli argument parsing is done in two places : internal/command/meta.go:578 internal/command/arguments/view.go:21
I can do a PR for this issue tomorrow
@Hefeweizen, `-show-all-warnings` seems too long so maybe just `-all-warnings` I'm unsure about the utility of `-show-similiar=5`, if you want to show more warnings without consolidation, why keep a limit ?