-
Notifications
You must be signed in to change notification settings - Fork 134
Added delete_namespace docs
#1339
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.
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
Conversation
Reviewer's GuideThis PR updates the namespaces guide by closing an unclosed code block and appending comprehensive documentation for the File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deploying datachain-documentation with
|
| Latest commit: |
0faaf95
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://573c9782.datachain-documentation.pages.dev |
| Branch Preview URL: | https://ilongin-1338-namespaces-docs.datachain-documentation.pages.dev |
docs/guide/namespaces.md
Outdated
| ``` | ||
|
|
||
| - **`<namespace>`** — deletes the namespace (must contain no projects or datasets) | ||
| - **`<namespace>.<project>`** — deletes the project (must contain no datasets) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check the layout (periods), etc
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1339 +/- ##
=======================================
Coverage 87.57% 87.57%
=======================================
Files 156 156
Lines 14502 14502
Branches 2083 2083
=======================================
Hits 12700 12700
Misses 1331 1331
Partials 471 471
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
@ilongin tests fail |
|
Also, even though 12 tests failed, I could see squash and merge cc @0x2b3bfa0 |
Added docs for
delete_namespaceand fix current docs formatting.Summary by Sourcery
Add documentation for the
delete_namespacefunction in the namespaces guide, including its signature, behavior, and usage examples, and correct existing formatting issues.Enhancements:
Documentation:
delete_namespacefunction with details on removing empty namespaces or projects