feat: Add project delete command to CLI by crawfordxx · Pull Request #6212 · feast-dev/feast
What this PR does / why we need it:
Exposes project deletion via the CLI. The registry already supports delete_project() programmatically, but there was no corresponding CLI command. This adds:
FeatureStore.delete_project(name)— thin wrapper aroundregistry.delete_project()feast projects delete <name>CLI command with a confirmation prompt--yes / -yflag to skip the confirmation (useful for scripting)
Which issue(s) this PR fixes:
Fixes #5095
Checks
- I've made sure the tests are passing.
- My commits are signed off (
git commit -s) - My PR title follows conventional commits format
Testing Strategy
- Manual tests
- Unit tests
Misc
Usage:
# Interactive (prompts for confirmation) feast projects delete my_project # Non-interactive feast projects delete my_project --yes
Expose project deletion via `feast projects delete <name>` CLI command. The registry already supports deleting projects programmatically, but there was no CLI command for it. - Add `delete_project` method to `FeatureStore` - Add `feast projects delete` CLI command with confirmation prompt - Support `--yes/-y` flag to skip confirmation Fixes #5095 Signed-off-by: majianhan <majianhan@kylinos.cn>
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
| commit: Whether the change should be persisted immediately. | ||
|
|
||
| Raises: | ||
| ProjectObjectNotFoundException: The project could not be found. |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ProjectObjectNotFoundException: The project could not be found. | |
| ProjectNotFoundException: The project could not be found. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters