-
Notifications
You must be signed in to change notification settings - Fork 513
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 include_named_queries_score to docs #6306
Add include_named_queries_score to docs #6306
Conversation
Signed-off-by: Dharin Shah <Dharin-shah@users.noreply.github.com>
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.
@kolchfa-aws Just a global comment on the table re: whether the parameter names and "true" and "false" should be in code font. Otherwise, LGTM.
@Dharin-shah Is this applicable to 2.12? |
@natebower I'll leave it for now because we have lots of instances like this and I'd like to fix all of them together some time. |
no, 3.0.0 and beyond. Also should we wait for the main feature PR to be merged before merging this? i need to make some changes based on some review comments |
@Dharin-shah Yes, we will wait until the code is merged to merge this PR. Thanks! |
We can merge this as well now :) |
Signed-off-by: Dharin Shah <Dharin-shah@users.noreply.github.com> Co-authored-by: Dharin Shah <Dharin-shah@users.noreply.github.com>
Closes #6278
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.