-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add reply sections to command pages #183
Conversation
Note: valkey-io/valkey-doc#202 is necessary to make this work |
They are built from resp2_replies.json/resp3_replies.json from valkey-doc. If the RESP2/RESP3 replies are equal, only a single section is added. Signed-off-by: Simon Baatz <gmbnomis@gmail.com>
972c5b5
to
a222e9f
Compare
@gmbnomis Thanks for the work on this! If this is dependent on a PR in another repo, could you please make this a draft PR so it doesn't get accidentally merged. |
I put this PR into draft state (and will wait until valkey-io/valkey-doc#202 gets merged) |
are we still waiting on valkey-io/valkey-doc#206 @gmbnomis ? |
@stockholmux No, the necessary PR was valkey-io/valkey-doc#202, which is merged. valkey-io/valkey-doc#206 just depends on this PR to become visible on the website. |
I did some spot checking and the output looks OK to me, if someone finds some obscure issues we can fix it. |
They are built from
resp2_replies.json
/resp3_replies.json
from valkey-doc. If the RESP2/RESP3 replies are equal, only a single section is added.Closes #185
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.