Anna Song

Results 97 comments of Anna Song

#4754 merged, so if you rebase your Lint error should go away.

/lgtm I don't have merge permissions, so will defer to @natasha41575. btw, great use of `types.Selector.String`!

@natasha41575 The PR binary works as expected on the following setup: `kustomization.yaml` ```yaml patchesStrategicMerge: - deploy.yaml - |- apiVersion: apps/v1 kind: Deployment metadata: name: nginx spec: template: spec: containers: -...

> > I know you didn't change [this line](https://github.com/kubernetes-sigs/kustomize/pull/4733/files#diff-8e36744c0d7d30f2d6ccd049742e599b37d63418c0242457a204e716f86ae58fR93), but could you add a newline to the end of it? > > Do you think [this commit](https://github.com/kubernetes-sigs/kustomize/pull/4733/files#diff-8e36744c0d7d30f2d6ccd049742e599b37d63418c0242457a204e716f86ae58fR93) fixes that? Yes,...

I can reproduce this issue. Same thing happens on v4.5.7 as well. My output is mostly the same, except the second resource is an empty string instead of being removed....

Hi @hiteshgakhreja1234, I don't believe you can currently `skip` in kustomize, but you can specify all the kinds under `configurations` you'd like to apply `namePrefix` to other than the 3...

Partially inspired by https://github.com/kubernetes-sigs/kustomize/pull/4652#discussion_r940546360, I removed the logic that calculates the destination path from `LocLoader`. This is because it violates a separation of concerns. I realized that as a result...