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

Some proofreading for the Dev Services for Kubernetes doc #29014

Merged
merged 2 commits into from Nov 3, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Nov 2, 2022

No description provided.

@quarkus-bot

This comment has been minimized.


In dev mode and when running tests, xref:kubernetes-dev-services.adoc[Dev Services for Kubernetes] automatically starts a Kubernetes API server.
In dev mode and when running tests, xref:kubernetes-dev-services.adoc[Dev Services for Kubernetes] automatically start a Kubernetes API server.
Copy link
Contributor

@geoand geoand Nov 3, 2022

Choose a reason for hiding this comment

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

I am not totally sure about this one. I understand why you made the change, but the way I read it is that Dev Services for Kubernetes is a single entity, therefore starts is appropriate. The same goes for the other changes made later on.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah I must admit I hesitated a bit about that one. So if you also do, let's keep it as is. I'll adjust the PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

@geoand it should be ready to go now.

@gsmet gsmet force-pushed the proofread-kubernetes-devservices branch from ad7878b to d779b5f Compare November 3, 2022 14:33
@gsmet gsmet force-pushed the proofread-kubernetes-devservices branch from d779b5f to 55ec341 Compare November 3, 2022 14:34
@quarkus-bot

This comment has been minimized.

Quarkus Documentation automation moved this from To do to Reviewer approved Nov 3, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Nov 3, 2022

Failing Jobs - Building 55ec341

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
✔️ JVM Tests - JDK 17
JVM Tests - JDK 17 MacOS M1 Set up runner ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 18

@geoand geoand merged commit 0e1f598 into quarkusio:main Nov 3, 2022
Quarkus Documentation automation moved this from Reviewer approved to Done Nov 3, 2022
@quarkus-bot quarkus-bot bot added this to the 2.15 - main milestone Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants