-
Notifications
You must be signed in to change notification settings - Fork 210
Init PipeCD Docs for PipeCD v1 #6316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Hey @khanhtc1202, The image (diagram) on the concepts page is supposed to be the same right? No changes to that in V1 I assume? |
|
Also, I was thinking to evaluate and test better, lets split this PR into
and so on. Let me know what you think. |
|
Also, I just realised we will have to update the index.html page as well. |
|
maybe we need some images from the overview plan blog? 👀 |
Yes, I don't think we have any changes on the overview, but if you want a better explain for piped plugins, maybe you can draw a separated diagram for piped<->plugin relationship. |
I was thinking of using this diagram from the overview blog: |
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
Signed-off-by: Eeshaan Sawant <eeshaans1@gmail.com>
khanhtc1202
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!

What this PR does:
Add Structure for PipeCD v1 docs
Why we need it:
Which issue(s) this PR fixes:
#6077
Fixes #
Does this PR introduce a user-facing change?: