Skip to content

Correct env.sh path in v16+ local examples docs#1569

Merged
mattlord merged 4 commits intoprodfrom
env_path_fix_v16+
Aug 28, 2023
Merged

Correct env.sh path in v16+ local examples docs#1569
mattlord merged 4 commits intoprodfrom
env_path_fix_v16+

Conversation

@mattlord
Copy link
Member

@mattlord mattlord commented Aug 28, 2023

We also simplify and unify the prompt usage in the commands:

  • Use $ as the static prompt
  • Only show the prompt when including the output (otherwise the commands are easier to cut and paste w/o it)

Fixes: #1523

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord requested a review from deepthi August 28, 2023 18:42
@netlify
Copy link

netlify bot commented Aug 28, 2023

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 857c36e
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/64ed11d817dd6700089dcce1
😎 Deploy Preview https://deploy-preview-1569--vitess.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>

```bash
source ./env.sh
source ../common/env.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

This was moved to common only in v17, so the v16 and v15 docs should not have any such change.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

Turns out the original PR was back ported to v16.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, here's the backport PR (bot failed): vitessio/vitess#12278

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord merged commit a7bda26 into prod Aug 28, 2023
@mattlord mattlord deleted the env_path_fix_v16+ branch August 28, 2023 21:32
@jfg956
Copy link
Contributor

jfg956 commented Aug 28, 2023

Only show the prompt when including the output (otherwise the commands are easier to cut and paste w/o it)

I like this, thanks.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vitess 17 Local Install instructions not working - bis.

3 participants