Skip to content

Conversation

astromechza
Copy link
Member

@astromechza astromechza commented May 20, 2025

Fixes #167. This PR improves the help text and guidance and adds a reference unit test to clarify how the delete ordering works and why it should be reversed.

…d delete

Signed-off-by: Ben Meier <ben.meier@humanitec.com>
mathieu-benoit
mathieu-benoit previously approved these changes May 20, 2025
Copy link
Contributor

@mathieu-benoit mathieu-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Ben Meier <ben.meier@humanitec.com>
Copy link
Contributor

@mathieu-benoit mathieu-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@astromechza astromechza disabled auto-merge May 20, 2025 15:05
@astromechza astromechza merged commit df89ee0 into main May 20, 2025
6 checks passed
@astromechza astromechza deleted the patching-tips branch May 20, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] Not possible to delete the default Deployment with a patch template
2 participants