okteto / docs

Apache License 2.0
6 stars 17 forks source link

change okteto destroy --all option description to fit new behavour #776

Closed teresaromero closed 2 months ago

teresaromero commented 3 months ago

Related to this change https://github.com/okteto/app/pull/8275

okteto destroy --all triggers the namespace-destroy-all which did not had the annotation skip logic. With the change this is introduce so the flag now will skip the resources with the annotation as described on the PR

netlify[bot] commented 3 months ago

Deploy Preview for okteto-docs ready!

Name Link
Latest commit 28ca8c64c8f0f0aa39c252b88c359c376b04fb28
Latest deploy log https://app.netlify.com/sites/okteto-docs/deploys/668fd22057e2fa0008dc74b5
Deploy Preview https://deploy-preview-776--okteto-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

ifbyol commented 3 months ago

Be aware that we need to wait until 1.22 release is done to merge this, as this won't be part of the ongoing release

codyjlandstrom commented 2 months ago

Marking as draft to prevent accidental merge

ifbyol commented 2 months ago

This can now be merged as we already merged the 1.22 release docs cc @teresaromero @codyjlandstrom