Skip to content

Split examples into v0 and v1 - #7173

Open
rahulshendre wants to merge 3 commits into
pipe-cd:masterfrom
rahulshendre:examples-restructure-v0-v1
Open

Split examples into v0 and v1#7173
rahulshendre wants to merge 3 commits into
pipe-cd:masterfrom
rahulshendre:examples-restructure-v0-v1

Conversation

@rahulshendre

Copy link
Copy Markdown
Contributor

What this PR does:
Restructures the examples/ directory into v0/ and v1/ subdirectories. All the current examples move under v0/, and I added a v1/ directory for the upcoming PipeCD v1 (plugin architecture) examples. The shared README.remote.md at the root now points to both, and the existing catalog moved into v0/README.md.

Why we need it:
Start of adding v1 examples and a lot of users still reference the v0 ones, so we keep both side by side instead of replacing them.

Which issue(s) this PR fixes:

Related to #6266

Does this PR introduce a user-facing change?:

  • How are users affected by this change:
  • Is this breaking change:
  • How to migrate (if breaking change):

Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
@rahulshendre
rahulshendre requested review from a team as code owners August 13, 2026 08:29
@rahulshendre rahulshendre changed the title split examples into v0 and v1 Split examples into v0 and v1 Aug 13, 2026

@aamirpatel aamirpatel 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.

Reviewed

@armistcxy armistcxy 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.

agree, split them to avoid confuse

@netlify

netlify Bot commented Aug 15, 2026

Copy link
Copy Markdown

Deploy Preview for pipecd-site canceled.

Name Link
🔨 Latest commit ad046e5
🔍 Latest deploy log https://app.netlify.com/projects/pipecd-site/deploys/6a800f89f1aafb00086ceabc

@armistcxy

Copy link
Copy Markdown
Contributor

@rahulshendre test fail due to moving examples, I think you should update the test also

Suggestion from Copilot when explaining action error

image

@armistcxy armistcxy 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.

I think we need to update docs/content/en/docs-dev/examples/_index.md also @rahulshendre

Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
@rahulshendre

Copy link
Copy Markdown
Contributor Author

I think we need to update docs/content/en/docs-dev/examples/_index.md also @rahulshendre

thanks for the mention, @armistcxy. those links will need updating, they point to the separate pipe-cd/examples repo (not the examples/ dir here), so they still resolve fine today and would only break if I switch them to v0/ before that repo is restructured. Per #6266 we're restructuring pipe-cd/examples the same way, and I'll update index.md to v0/ as part of that.
I was thinking of keeping this PR scoped to the main-repo move, what do you think 👀

@rahulshendre

Copy link
Copy Markdown
Contributor Author

@rahulshendre test fail due to moving examples, I think you should update the test also

Thanks @armistcxy, nice catch. That test hardcodes the old examples/kubernetes/simple path, so the move to examples/v0/ broke it. I've updated both paths in sync_test.go to point at examples/v0/. PTAL

@armistcxy

Copy link
Copy Markdown
Contributor

I was thinking of keeping this PR scoped to the main-repo move, what do you think 👀

Agree, thanks for pointing out those links belong to another repo

@rahulshendre

Copy link
Copy Markdown
Contributor Author

@khanhtc1202 @mohammedfirdouss, PTAL when you have time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants