We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Currently we have two UI options:
But the ui.UI defines only one method: Display(ctx context.Context, disks unused.Disks, extraColumns []string) error. Here clearly the verbose mode option is missing, and we're relying on passing that to the different ui.UI implementation constructors.
ui.UI
Display(ctx context.Context, disks unused.Disks, extraColumns []string) error
In the future we also might introduce more UI options, so let's do the following changes:
ui.Options
UI.Display
Display(context.Context, ui.Options, unused.Disks) error
The text was updated successfully, but these errors were encountered:
This was addressed in #19.
Sorry, something went wrong.
No branches or pull requests
Currently we have two UI options:
But the
ui.UI
defines only one method:Display(ctx context.Context, disks unused.Disks, extraColumns []string) error
. Here clearly the verbose mode option is missing, and we're relying on passing that to the differentui.UI
implementation constructors.In the future we also might introduce more UI options, so let's do the following changes:
ui.Options
or similar struct to hold these configurationsUI.Display
signature to something likeDisplay(context.Context, ui.Options, unused.Disks) error
The text was updated successfully, but these errors were encountered: