gerlowskija commented on PR #685: URL: https://github.com/apache/solr-operator/pull/685#issuecomment-2528643576
Hi @smoldenhauer-ish - this LGTM overall. I did try to push one minor change to your branch, to reword the `helm/solr-operator/Chart.yaml` entry slightly. But it looks like I don't have the requisite permissions to collaborate on that branch. If you're willing to share write-access on the PR branch, I'll try sharing again? (Or alternately, you could incorporate the ["Suggested Change" here](https://github.com/apache/solr-operator/pull/685/files#r1876310454).) Otherwise this is ready to merge IMO. I'll hold off for a bit in hopes you see this and answer, but won't block merging the PR too long, as it's just a minor doc tweak. -- 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
