docs: add instructions for stopping Flyte sandbox#7029
docs: add instructions for stopping Flyte sandbox#7029muskan-creates352 wants to merge 2 commits intoflyteorg:masterfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7029 +/- ##
=======================================
Coverage 56.96% 56.97%
=======================================
Files 931 931
Lines 58276 58276
=======================================
+ Hits 33198 33201 +3
+ Misses 22019 22016 -3
Partials 3059 3059
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
0d800d5 to
86ebf65
Compare
|
Hi sir, I have fixed the DCO issue and updated the commit. Please let me know if any further changes are needed. Thank you! |
|
Hi, Whenever you have some time, I would be grateful if you could kindly review it. Please let me know if any changes are needed from my side. Thank you so much for your time and guidance! |
|
Hi, thanks for contributing. Please fix the DCO test |
6ab32eb to
e5fc24c
Compare
|
It seems like a bunch of unintentional changes made it into here. @muskan-creates352 can you clean that up as well as the DCO? |
e5fc24c to
3e0d18e
Compare
|
Hi, thanks for your feedback! I've cleaned up the PR and ensured that only the intended documentation change is included. I've also fixed the DCO issue. Please let me know if any further changes are needed. |
|
Hi, the PR has been cleaned up and rebased on the latest upstream. All checks have passed. Could you please review when you get a chance? Thanks! |
|
Hi, I have updated the branch with the latest changes from the base branch. All checks are passing on my side. Could you please review it when you get a chance? Thanks a lot! |
43cd7c2 to
5a36d38
Compare
Signed-off-by: Muskan Kumari <er.muskan09@gmail.com>
fb36b8e to
6cbd54b
Compare
Tracking issue
Why are the changes needed?
->While exploring the sandbox deployment documentation, I noticed that the command to stop the Flyte sandbox was not mentioned. This can be helpful for users who want to stop the local sandbox after testing.
What changes were proposed in this pull request?
->This PR adds a small section in sandbox.rst explaining how to stop the Flyte sandbox using the command:
flytect1 demo stop
This will help users working with the local Flyte sandbox environment.
How was this patch tested?
->The documentation change was verified by reviewing the updated section in sandbox.rst.
Labels
Please add one or more of the following labels to categorize your PR:
This is important to improve the readability of release notes.
Setup process
Screenshots
Check all the applicable boxes
Related PRs
Docs link