Skip to content

follow Kubebuilder recommendations in maintenance#296

Merged
satoru-takeuchi merged 2 commits into
mainfrom
push-mumuzkuksspl
Jun 22, 2026
Merged

follow Kubebuilder recommendations in maintenance#296
satoru-takeuchi merged 2 commits into
mainfrom
push-mumuzkuksspl

Conversation

@ushitora-anqou

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s maintenance guidance (and Go module metadata) to align version selection with Kubebuilder’s compatibility recommendations.

Changes:

  • Adjusts go.mod Go version declaration.
  • Updates docs/maintenance.md to direct maintainers to Kubebuilder testdata for compatible dependency/Go versions and refines upgrade steps wording.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
go.mod Updates the declared Go version (and removes the toolchain pin).
docs/maintenance.md Revises maintenance instructions to follow Kubebuilder compatibility guidance and clarifies upgrade steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread go.mod
Comment thread docs/maintenance.md Outdated
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Kubebuilder uses go 1.25.7, so we should use it, too.

cf. https://github.com/kubernetes-sigs/kubebuilder/blob/v4.14.0/testdata/project-v4/go.mod#L3

Note that, if we try to stick to the versions Kubebuilder is using, we
should change k8s.io/* libraries' versions from v0.35.4 to v0.35.2.
However, there is no strong reason to downgrade the libraries because
the current version of Mantle is stable. Thus, for now, we'll just
follow the Go version only. Once we need to support newer versions of
Kubernetes we'll follow the Kubebuilder's recommendetaions strictly.

Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
@ushitora-anqou ushitora-anqou marked this pull request as ready for review June 22, 2026 00:44
@ushitora-anqou ushitora-anqou requested a review from a team as a code owner June 22, 2026 00:44
@ushitora-anqou ushitora-anqou requested review from satoru-takeuchi and removed request for a team June 22, 2026 00:44
@satoru-takeuchi satoru-takeuchi merged commit 6914aa0 into main Jun 22, 2026
15 checks passed
@satoru-takeuchi satoru-takeuchi deleted the push-mumuzkuksspl branch June 22, 2026 01:31
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.

3 participants