Skip to content
This repository has been archived by the owner on Jul 10, 2021. It is now read-only.

Update index.md #1967

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions guides/user/pipeline/pipeline-templates/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ on an existing pipeline.

* Share the template with one or more teams of developers using Spinnaker.

To [save a pipeline
[Saving a pipeline
Copy link
Contributor

Choose a reason for hiding this comment

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

The rest of the list starts w/ a verb, such as use. Can you rewrite the sentence to be start w/ "Save" so that it's consistent with the others?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi Brian,

The rest of the list does start with a verb, but I thought this sentence in particular is more of a note to provide additional context to the second bullet. If this sentence should be an action / step, then I think this should appear as the bullet instead.

Ex.

  • Save a pipeline template to Spinnaker.

This will make the template available to developers. It's a good idea to communicate to the team about what templates are available.

template](/guides/user/pipeline/pipeline-templates/create/#4-save-the-template)
to Spinnaker is to make it available to developers. It's a good idea though
to Spinnaker makes it available to developers. It's a good idea
to communicate to the team what templates are available.

* [Use the `spin` CLI to plan how to parameterize the
Expand Down