Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixStuffDueToKubectlDeletion #3

Merged

Conversation

monopole
Copy link
Contributor

@monopole monopole commented May 15, 2018

Demos broke when the demo data in the kubectl repo was deleted. Tests WAI, yay

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 15, 2018
@monopole monopole merged commit 831bd5d into kubernetes-sigs:master May 15, 2018
@monopole monopole deleted the fixStuffDueToKubectlDeletion branch June 8, 2018 15:45
@tkellen tkellen mentioned this pull request Oct 16, 2019
yanniszark added a commit to arrikto/kubernetes-sigs-kustomize that referenced this pull request Apr 13, 2021
Commit 6b99caa79 tried to fix glob handling but broke normal paths.
Fix that.

GitHub-PR: kubernetes-sigs#3.

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
Reviewed-by: Dimitris Aragiorgis <dimara@arrikto.com>
annasong20 added a commit to annasong20/kustomize that referenced this pull request Aug 17, 2022
annasong20 added a commit to annasong20/kustomize that referenced this pull request Aug 17, 2022
k8s-ci-robot pushed a commit that referenced this pull request Aug 19, 2022
* Implement file localization for localize

Implement file localization for new command kustomize localize

* Patch file localization

Fix lint errors, address feedback, begin transition to file loader from
localizer

* Add Repo() to Loader

* Implement locLoader factory + cleanup

* Add domain to RepoSpec

Fix ssh relative url host parsing for non-github domain on the side

* Implement Load(), New(), Root(), Dst() for LocLoader

* Address repospec code review comments

* Address 1st round of code review feedback

* Address feedback #2

Removed localized path calculations to cover edge case and improve readability.

* Remove ldr Cleanup logging

* Address code review round #3

* Address code review feedback #4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants