Skip to content

feat: implement organization context in the cli #12259

New issue

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

Sign up for GitHub

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

Merged
merged 15 commits into from
Feb 26, 2024
Merged

feat: implement organization context in the cli #12259

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
31a8541
feat: begin work to implement switching orgs in the cli
Emyrk Feb 21, 2024
1ac16cf
add only-id option
Emyrk Feb 21, 2024
6a99a37
Mark hidden
Emyrk Feb 22, 2024
d88d7be
linting
Emyrk Feb 22, 2024
5df951b
fix test assert
Emyrk Feb 22, 2024
a2f68ea
linting
Emyrk Feb 22, 2024
50cdb32
Lint
Emyrk Feb 22, 2024
438e69d
Add unit test
Emyrk Feb 22, 2024
f32c78c
linting
Emyrk Feb 22, 2024
cad75c2
fix use
Emyrk Feb 22, 2024
f8b97d9
implement more general organizations show
Emyrk Feb 23, 2024
405afc9
fix short desc
Emyrk Feb 23, 2024
a9da679
fix help msg
Emyrk Feb 23, 2024
510dbb8
Fix test
Emyrk Feb 23, 2024
c74106b
whoops, fix the unit test
Emyrk Feb 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Mark hidden
  • Loading branch information
@Emyrk
Emyrk committed Feb 22, 2024
commit 6a99a37a533ab7df1c622d88c7a702bad30c285d
4 changes: 2 additions & 2 deletions cli/organization.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ func (r *RootCmd) organizations() *clibase.Cmd {
Use: "organizations { current }",
Short: "Organization related commands",
Aliases: []string{"organization", "org", "orgs"},

Hidden: true, // Hidden until these commands are complete.
Copy link
Member Author

Choose a reason for hiding this comment

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

I marked this as hidden, I think we can defer the actual command name discussion and get the code in. Don't want to bikeshed and prevent code progress.

Handler: func(inv *clibase.Invocation) error {
return inv.Command.HelpHandler(inv)
},
Expand Down Expand Up @@ -45,7 +45,7 @@ func (r *RootCmd) currentOrganization() *clibase.Cmd {
)
cmd := &clibase.Cmd{
Use: "current",
Short: "Show the current selected organization the cli will use",
Short: "Show the current selected organization the cli will use.",
Middleware: clibase.Chain(
r.InitClient(client),
),
Expand Down