-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
docs: add consul_kv to the README.md file #8865
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you remove the unrelated changes?
Signed-off-by: Amishakumari544 <amishakumari544@gmail.com>
587d642
to
b1fdf96
Compare
Signed-off-by: Amishakumari544 <amishakumari544@gmail.com>
apologies , done now. @navendu-pottekkat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if this needs to be added. Maybe others can comment.
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@apisix.apache.org list. Thank you for your contributions. |
@Amishakumari544 it will be great if you can help to resolve conflicts, thanks |
@Amishakumari544 Could you fix the CI error? |
@kayx23 please take a look at this PR, thx |
Description
Fixes #8533
Checklist
cc- @spacewander @bzp2010 @moonming @navendu-pottekkat