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
The documentation command should not be instantiated for every command supercommand. Every binary or sub-super command that implements the super command now has a documentation command. This includes things like jujud, container-agent, etc.
Instead, a documentation command should be added to the super command base as an adapter. Ideally, the documentation command should be in a sub-package, so that it can be used in isolation.
An example of this should be:
waitFor := cmd.NewSuperCommand(cmd.SuperCommandParams{
Name: "wait-for",
UsagePrefix: "juju",
Doc: waitForDoc,
Purpose: "Wait for an entity to reach a specified state.",
Documentation: documentation.NewMarkdown(),
})
If there is no documentation, it doesn't do anything. This is purely additive.
The text was updated successfully, but these errors were encountered:
This is another breaking change.
The documentation command should not be instantiated for every command supercommand. Every binary or sub-super command that implements the super command now has a documentation command. This includes things like jujud, container-agent, etc.
Instead, a documentation command should be added to the super command base as an adapter. Ideally, the documentation command should be in a sub-package, so that it can be used in isolation.
An example of this should be:
If there is no documentation, it doesn't do anything. This is purely additive.
The text was updated successfully, but these errors were encountered: