Skip to content

Recomment out API replace for v1.1.1-pre.1 - #4792

Merged
mozesl-nokia merged 1 commit into
kptdev:mainfrom
nokia:api-unreplace
Oct 7, 2026
Merged

mozesl-nokia merged 1 commit into
kptdev:mainfrom
nokia:api-unreplace

Conversation

@mozesl-nokia

Copy link
Copy Markdown
Contributor

Following up on #4774, remove the replace directive from go.mod and use the new API module prerelease.

Signed-off-by: Mózes László Máté <laszlo.mozes@nokia.com>
@mozesl-nokia
mozesl-nokia requested review from a team and rendre-greyling and a balanced review from Copilot October 7, 2026 13:14
@mozesl-nokia mozesl-nokia added the dependencies Pull requests that update a dependency file label Oct 7, 2026
@mozesl-nokia mozesl-nokia added area/release kpt release issues area/fn-runtime KRM function runtime area/api labels Oct 7, 2026
@netlify

netlify Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit 2152885
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6ac6459de6a11f0008ad2215
😎 Deploy Preview https://deploy-preview-4792--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Human review and CI should confirm the published prerelease includes the required PreserveNulls API changes.

Review effort: Balanced
Findings: None

What changed in this PR

Switches kpt to the published API prerelease following #4774, replacing the temporary local-module override.

Changes:

  • Comments out the local API replacement and selects v1.1.1-pre.1.
  • Updates the matching module checksums.
File Description
go.sum Records API prerelease checksums.
go.mod Disables the local replacement and updates the API dependency.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@mozesl-nokia
mozesl-nokia merged commit 015dfa8 into kptdev:main Oct 7, 2026
27 checks passed
@mozesl-nokia
mozesl-nokia deleted the api-unreplace branch October 7, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api area/fn-runtime KRM function runtime area/release kpt release issues dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants