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

Fix current compilation error #1294

Merged
merged 36 commits into from
Apr 8, 2023

Conversation

wind57
Copy link
Contributor

@wind57 wind57 commented Apr 8, 2023

No description provided.

wind57 and others added 30 commits December 4, 2021 07:59
@wind57 wind57 marked this pull request as ready for review April 8, 2023 09:00
KubernetesEnvironmentRepository environmentRepository = new KubernetesEnvironmentRepository(coreApi,
kubernetesPropertySourceSuppliers, "default");
KUBERNETES_PROPERTY_SOURCE_SUPPLIER, "default");
Copy link
Contributor Author

Choose a reason for hiding this comment

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

KUBERNETES_PROPERTY_SOURCE_SUPPLIER this was missing, probably a small merge issue. The rest is formatting, nothing else

@ryanjbaxter ryanjbaxter merged commit 25b9d96 into spring-cloud:main Apr 8, 2023
@ryanjbaxter
Copy link
Contributor

Thanks, I merged some stuff forward and messed it all up. I should not have done this on a Friday afternoon.

@wind57
Copy link
Contributor Author

wind57 commented Apr 8, 2023

debatable if you should not have done it (joking, of course) :) I was very eager to get some of these small PRs in, before I present some really meaningful work to you. Thank you for merging it

@ryanjbaxter
Copy link
Contributor

😂 it's not debatable I definitely shouldn't have done it.

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants