Rewrite the installation guide for clarity and flow. #2104
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Rewrites the Tekton Pipelines installation guide for better clarity and flow.
googlebot
added
the
cla: yes
tekton-robot
added
the
size/L
|
/kind documentation |
tekton-robot
added
the
kind/documentation
ghost
approved these changes
Feb 26, 2020
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbwsg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
tekton-robot
added
the
approved
Fixes typos and applies the correct verb to "ConfigMap."
|
/retest
…On Wed, Feb 26, 2020 at 4:41 PM tekton-robot ***@***.***> wrote:
@spomorski <https://github.com/spomorski>: The following tests *failed*,
say /retest to rerun them all:
Test name Commit Details Rerun command
pull-tekton-pipeline-go-coverage e99818d
<e99818d>
link
<https://tekton-releases.appspot.com/build/tekton-prow/pr-logs/pull/tektoncd_pipeline/2104/pull-tekton-pipeline-go-coverage/1232782578843389952/> /test
pull-tekton-pipeline-go-coverage
pull-tekton-pipeline-build-tests e99818d
<e99818d>
link
<https://tekton-releases.appspot.com/build/tekton-prow/pr-logs/pull/tektoncd_pipeline/2104/pull-tekton-pipeline-build-tests/1232782578843389953/> /test
pull-tekton-pipeline-build-tests
pull-tekton-pipeline-unit-tests e99818d
<e99818d>
link
<https://tekton-releases.appspot.com/build/tekton-prow/pr-logs/pull/tektoncd_pipeline/2104/pull-tekton-pipeline-unit-tests/1232782578843389954/> /test
tekton-pipeline-unit-tests
pull-tekton-pipeline-integration-tests e99818d
<e99818d>
link
<https://tekton-releases.appspot.com/build/tekton-prow/pr-logs/pull/tektoncd_pipeline/2104/pull-tekton-pipeline-integration-tests/1232782578843389955/> /test
pull-tekton-pipeline-integration-tests
Full PR test history
<https://tekton-releases.appspot.com/pr/tektoncd_pipeline/2104>. Your PR
dashboard <https://prow.tekton.dev/pr/spomorski>.
Instructions for interacting with me using PR comments are available here
<https://git.k8s.io/community/contributors/guide/pull-requests.md>. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
<https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:>
repository. I understand the commands that are listed here
<https://go.k8s.io/bot-commands>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2104?email_source=notifications&email_token=AG3UR4TXSYKEBUMHP2JGSWLRE3O2BA5CNFSM4K3PWPJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENB77IQ#issuecomment-591658914>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG3UR4XRPYO67QTJDM24GLTRE3O2BANCNFSM4K3PWPJA>
.
|
Since my rewrite is a lot more extensive, it's better to integrate your addition into my PR. |
Addresses comments from afrittoli@.
Adds the GoogleCloudStorage example config and other changes from PR 2123.
bobcatfish
reviewed
Mar 3, 2020
docs/install.md
Outdated
|
|
||
| * **[Official](https://github.com/tektoncd/pipeline/releases)** - install this unless you have | ||
| a specific reason to go for a different release. | ||
|
|
There was a problem hiding this comment.
extra newline?
I AM SO SORRY
Fixes formatting.
|
/hold for commit squash :D |
tekton-robot
added
the
do-not-merge/hold
tekton-robot
pushed a commit
that referenced
this pull request
Mar 3, 2020
Rewrites the Tekton Pipelines Installation Guide for improved clarity, organization, and flow. This is a commit-squashed version of #2104.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rewrites the Tekton Pipelines installation guide for better clarity and flow.