Skip to content

add namespace delete command#158

Merged
mattkim merged 10 commits intomainfrom
mattkim/delete-namespace
Mar 29, 2023
Merged

add namespace delete command#158
mattkim merged 10 commits intomainfrom
mattkim/delete-namespace

Conversation

@mattkim
Copy link
Copy Markdown
Member

@mattkim mattkim commented Mar 23, 2023 •
edited
Loading

What was changed

add namespace delete command

Why?

allow users to delete a namespace using tcld

Checklist

  1. How was this tested:

unit tests

mastermanu
mastermanu reviewed Mar 28, 2023
View reviewed changes
Comment thread app/namespace.go
@mattkim mattkim merged commit 636b69a into main Mar 29, 2023
@mattkim mattkim deleted the mattkim/delete-namespace branch March 29, 2023 22:03
tminusplus added a commit that referenced this pull request Apr 17, 2023
@tminusplus @mattkim
Merge latest changes from main into release/apikeys (#171)
6bb2dcb 
* remove config on logout (#149)

* add create namespace command (#151)

* add namespace delete command (#158)

* bump version in tcld to v0.5.0 (#163)

* Tighten permissions on config files (#170)

---------

Co-authored-by: mattkim <matt@temporal.io>
tminusplus added a commit that referenced this pull request Apr 25, 2023
@tminusplus @mattkim
Merge latest changes from main into release/apikeys (#171)
8f49632 
* remove config on logout (#149)

* add create namespace command (#151)

* add namespace delete command (#158)

* bump version in tcld to v0.5.0 (#163)

* Tighten permissions on config files (#170)

---------

Co-authored-by: mattkim <matt@temporal.io>
@rachfop rachfop mentioned this pull request May 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mastermanu mastermanu mastermanu approved these changes

@anekkanti anekkanti anekkanti approved these changes

@feihuang feihuang Awaiting requested review from feihuang

@tminusplus tminusplus Awaiting requested review from tminusplus

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mattkim @mastermanu @anekkanti