Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix tenant flags for tenant variables list and tenant view #299

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

thampton
Copy link

@thampton thampton commented Nov 30, 2023

Partial fix for #297

Adds the --tenant flag as shown in the documentation for

octopus tenant variables list --tenant "example tenant"

Adds the --tenant flag as shown in the documentation for

octopus tenant view --tenant "example tenant"

@thampton thampton changed the title Fix tenant flags for tenant variables Fix tenant flags for tenant variables list and tenant view Nov 30, 2023
Comment on lines -33 to -40
type ViewOptions struct {
Client *client.Client
Host string
out io.Writer
idOrName string
flags *ViewFlags
}

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@domenicsim1 I removed this struct because it seemed simplier without it. Should this be retained for unit test purposes?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant