Skip to content

docs: updates for PR #4405#4412

Merged
efiacor merged 4 commits intomainfrom
dosu/doc-update-4405
Mar 5, 2026
Merged

docs: updates for PR #4405#4412
efiacor merged 4 commits intomainfrom
dosu/doc-update-4405

Conversation

@dosubot
Copy link
Contributor

@dosubot dosubot bot commented Feb 25, 2026

Update documentation for #4405

Generated by Dosu

Copilot AI review requested due to automatic review settings February 25, 2026 08:31
@dosubot dosubot bot review requested due to automatic review settings February 25, 2026 08:31
@netlify
Copy link

netlify bot commented Feb 25, 2026

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit f9f4f4d
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/69a87a7ca9ec3400089d82d7
😎 Deploy Preview https://deploy-preview-4412--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. documentation Improvements or additions to documentation labels Feb 25, 2026
@liamfallon
Copy link
Contributor

This auto-generated PR contains documentation changes for PRs not merged yet: ie #4411

@CsatariGergely
Copy link
Contributor

Please resolve the conflicts

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
Copilot AI review requested due to automatic review settings February 26, 2026 07:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates documentation to reflect the new auto-create directory feature introduced in PR #4405, which enables kpt pkg init to automatically create package directories.

Changes:

  • Updated documentation examples to remove manual mkdir steps before kpt pkg init
  • Clarified CI best practices regarding when not to use kpt pkg init
  • Added example output showing the directory creation message

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
local-config/_index.md Updated example to use kpt pkg init my-pkg instead of mkdir + kpt pkg init workflow
depends-on/_index.md Updated example to use kpt pkg init my-pkg instead of mkdir + kpt pkg init workflow
apply-time-mutation/_index.md Updated example to use kpt pkg init my-pkg instead of mkdir + kpt pkg init workflow
migration.md Updated migration guide description (note: contains trailing whitespace)
namespace-provisioning-cli.md Updated guide with new workflow and example output showing directory creation
08-ci-user-guide/_index.md Clarified anti-pattern to specify not creating package metadata in CI
03-packages/_index.md Contains unresolved merge conflicts between two versions describing package creation
02-concepts/_index.md Contains unresolved merge conflicts in Kptfile Annotations section

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

Signed-off-by: Gergely Csatari <gergely.csatari@nokia.com>
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Mar 4, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Contributor

@liamfallon liamfallon left a comment

Choose a reason for hiding this comment

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

Please resolve the patch comments.

@dosubot dosubot bot removed the lgtm label Mar 4, 2026
@dosubot dosubot bot added the lgtm label Mar 5, 2026
@liamfallon liamfallon self-requested a review March 5, 2026 10:44
@efiacor efiacor merged commit c629fdd into main Mar 5, 2026
13 checks passed
@efiacor efiacor deleted the dosu/doc-update-4405 branch March 5, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants