Skip to content

docs: fix simple typos#6990

Closed
lpmi-13 wants to merge 1 commit into
operator-framework:masterfrom
lpmi-13:dockerfile-comments-typofix
Closed

docs: fix simple typos#6990
lpmi-13 wants to merge 1 commit into
operator-framework:masterfrom
lpmi-13:dockerfile-comments-typofix

Conversation

@lpmi-13
Copy link
Copy Markdown

@lpmi-13 lpmi-13 commented Aug 25, 2025

Description of the change:

Fixes two simple typos in the comments of the Dockerfiles.

Motivation for the change:

I noticed in my generated Dockerfile after calling operator-sdk create-api that there were a couple simple typos, and the only place I could find that string was in these files. I'm unsure whether this will also impact the generated Dockerfiles for the project, but if that's also in this repo somewhere, I would happily add that change to this PR as well.

Checklist

N/A

If the pull request includes user-facing changes, extra documentation is required:

@openshift-ci openshift-ci Bot requested review from grokspawn and joelanford August 25, 2025 08:04
Signed-off-by: Adam Leskis <leskis@gmail.com>
@lpmi-13 lpmi-13 force-pushed the dockerfile-comments-typofix branch from 551dfc4 to 142db94 Compare August 25, 2025 08:05
@acornett21
Copy link
Copy Markdown
Contributor

Hi @lpmi-13 These files actually come from kubebuilder you'd need to raise a PR to that repo to fix this at the source, since this would get overwritten at testing/generation time.

I believe the below is the source.

https://github.com/kubernetes-sigs/kubebuilder/blob/master/pkg/plugins/golang/v4/scaffolds/internal/templates/dockerfile.go#L60

@acornett21 acornett21 closed this Aug 26, 2025
@lpmi-13
Copy link
Copy Markdown
Author

lpmi-13 commented Aug 26, 2025

Much obliged, I'll raise something there instead 🫡

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.

2 participants