joshgog commented on PR #1061: URL: https://github.com/apache/solr/pull/1061#issuecomment-1276932256
> 1. I noticed you left the `./gradlew check` box unchecked on the checklist in the PR description. Running `check` isn't strictly required, as most committers will run it themselves before merging code. But it does help give your reviewer a bit of extra confidence as it signals: "Hey, the code quality checks and tests and all that passed for me, so this should be ready to review". So if you're willing, feel free to give that a try when you get a few minutes and let me know how it went. There was one failed test. JWT security issues. I'm rerunning the check to note the details of the error. Where is the test log located? To avoid rerunning the check. > 2\. End-user-visible changes in Solr usually get mentioned in a "change-log" file that lives at `solr/CHANGES.txt`. This file exists both to give users an idea of what might've changed release-to-release, but also as a way to give credit and recognition to all the contributors like you who helped. If you get a few minutes, could you document this change by adding a bullet point under the "Improvements" section for the 9.2.0 release? Alternately, if you don't have time or aren't sure what to write, I'm happy to draft that myself if you can tell me what name you'd like to be credited under. (Some people use their full name, some people use a pseudonym or username, etc.) I have documented the changes. > 3\. Lastly, since we're changing a Solr API we should probably update the documentation anywhere it might be mentioned. Solr's main documentation, the "Solr Reference Guide" lives alongside the code in this repo at `solr/solr-ref-guide`. It can be built using the command `./gradlew buildLocalSite` - more detail on this in the README [here](https://github.com/apache/solr/blob/main/solr/solr-ref-guide/README.adoc). The one place in particular that I know needs updated is the section [here](https://github.com/apache/solr/blob/main/solr/solr-ref-guide/modules/deployment-guide/pages/cluster-node-management.adoc#clusterstatus-cluster-status). Could you take a crack at updating that and let me know how it goes? I have updated the reference guide -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
