Skip to content
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

sync cobradocs #1605

Merged
merged 4 commits into from
Sep 28, 2023
Merged

sync cobradocs #1605

merged 4 commits into from
Sep 28, 2023

Conversation

ajm188
Copy link
Contributor

@ajm188 ajm188 commented Sep 27, 2023

first half of #1602 (next step is to get this hooked up to some sort of on-push/merge hook from the main codebase side of things)

Andrew Mason added 2 commits September 27, 2023 12:07
Signed-off-by: Andrew Mason <andrew@planetscale.com>
Signed-off-by: Andrew Mason <andrew@planetscale.com>
@@ -24,7 +25,7 @@ This helps ensure that `mysqld` is automatically restarted after failures.
--config-file string Full path of the config file (with extension) to use. If set, --config-path, --config-type, and --config-name are ignored.
--config-file-not-found-handling ConfigFileNotFoundHandling Behavior when a config file is not found. (Options: error, exit, ignore, warn) (default warn)
--config-name string Name of the config file (without extension) to search for. (default "vtconfig")
--config-path strings Paths to search for config files in. (default [/Users/andrew/dev/vitess])
--config-path strings Paths to search for config files in. (default [/Users/andrew/dev/vtwebsite/vitessio.website/vitess])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unfortunately this is gonna generate some unnecessary churn until i get to #1601 (which i'm working on separately)

@netlify
Copy link

netlify bot commented Sep 27, 2023

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit b6cda52
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/6515547015859b00088a6923
😎 Deploy Preview https://deploy-preview-1605--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.

Copy link
Member

@GuptaManan100 GuptaManan100 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Andrew Mason <andrew@planetscale.com>
@@ -25,7 +25,7 @@ This helps ensure that `mysqld` is automatically restarted after failures.
--config-file string Full path of the config file (with extension) to use. If set, --config-path, --config-type, and --config-name are ignored.
--config-file-not-found-handling ConfigFileNotFoundHandling Behavior when a config file is not found. (Options: error, exit, ignore, warn) (default warn)
--config-name string Name of the config file (without extension) to search for. (default "vtconfig")
--config-path strings Paths to search for config files in. (default [/Users/andrew/dev/vtwebsite/vitessio.website/vitess])
--config-path strings Paths to search for config files in. (default [$WORKDIR])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

🎉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

note that the older versions will get this change when we update the release tags (since we currently "sync" to the v17.x.y (and so on for 16 and 15) tag instead of release-17.0

Signed-off-by: Andrew Mason <andrew@planetscale.com>
@ajm188 ajm188 merged commit 862c506 into prod Sep 28, 2023
4 checks passed
@ajm188 ajm188 deleted the andrew/sync-cobradocs branch September 28, 2023 10:28
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.

3 participants