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

Update index.md #1957

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Update index.md #1957

wants to merge 7 commits into from

Conversation

stephly07
Copy link
Contributor

Removed unnecessary punctuation marks and words
Removed finger pointing icon for consistency
Broke a sentence into two since it seems like two different thoughts

stephly07 and others added 3 commits July 21, 2020 15:31
Removed unnecessary punctuation marks and words
Removed finger pointing icon for consistency
Broke a sentence into two since it seems like two different thoughts
@@ -44,8 +44,9 @@ endpoint depends on how you've configured your [Spinnaker
endpoints](/setup/security) -- if you're running on a different endpoint, for
example `https://api.spinnaker-prod.net`, that'll be shown instead.

Keeping track of that endpoint as `$ENDPOINT` (it will depend on where
Spinnaker is installed), save that pipeline, and run:
Keeping track of that endpoint as `$ENDPOINT` will depend on where
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe just:
"Save that pipeline and run:
[command, including $ENDPOINT]
Where $ENDPOINT is the URL for the endpoint, which depends on where Spinnaker is installed."

Or something like that?

Copy link
Contributor

Choose a reason for hiding this comment

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

These would work better as imperative verbs in numbered steps, but you'd have to do that to the entire article.
This paragraph here is the third step in this section.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants