Skip to content

Conversation

@yolken-segment
Copy link
Contributor

Description

This change updates the cluster selection process when using globs with wildcards to prune out clusters with no changes. Otherwise, if a user comments with something like kubeapply apply *, this operation might get applied to clusters that weren't shown in the original diff.

Testing

Testing completed successfully via some additional unit tests.

Copy link

@achille-roussel achille-roussel left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix 🙌

@yolken-segment yolken-segment merged commit b45740f into master Sep 1, 2021
@yolken-segment yolken-segment deleted the yolken-fix-globs branch September 1, 2021 21:21
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.

3 participants