github-actions[bot] commented on PR #5325:
URL: https://github.com/apache/solr/pull/5325#issuecomment-6821121947

   **Automated PR readiness judgements** (LLM-generated; may be wrong; not 
updated on later pushes).
   
   - ✅ [Jira issue in 
title](https://github.com/apache/solr/blob/main/CONTRIBUTING.md): already 
present
   - ✅ 
[Changelog](https://github.com/apache/solr/blob/main/dev-docs/changelog.adoc#_changelog_types_reference):
 type: fixed <!-- The change fixes a bug where the 
&#96;SOLR&#95;PID&#95;DIR&#96; environment variable was ignored on Windows&#44; 
causing port files to be written to the installation directory instead of the 
specified location&#44; which contradicts the documentation and breaks 
&#96;stop &#45;&#45;all&#96;&#47;​&#96;status&#96; for users who set the 
variable&#46;​ -->
   - ✅ [Reference Guide 
update](https://github.com/apache/solr/tree/main/solr/solr-ref-guide): not 
needed <!-- The Reference Guide already documents that 
&#96;SOLR&#95;PID&#95;DIR&#96; is used by &#96;status&#96; and recommends 
setting it&#59; this PR simply makes the Windows script behave as already 
documented&#44; so no new documentation is required&#46;​ -->
   - ✅ [Upgrade 
notes](https://github.com/apache/solr/tree/main/solr/solr-ref-guide/modules/upgrade-notes/pages):
 not needed <!-- The change is backwards&#45;compatible&#59; the default value 
for &#96;SOLR&#95;PID&#95;DIR&#96; remains 
&#96;&#37;SOLR&#95;TIP&#37;&#92;bin&#96;&#44; so behavior is unchanged for 
users who do not explicitly set the variable&#46;​ -->
   - ✅ Tests: not needed <!-- The change is to a Windows batch script 
&#40;&#96;solr&#47;​bin&#47;​solr&#46;​cmd&#96;&#41;&#44; which is developer 
tooling&#47;​CLI infrastructure that is not covered by the standard Java test 
suite&#44; and the author notes that no automated test harness exists for it 
yet&#46;​ -->
   
   <!-- Assessed commit 81fd98a7c8e496c5d37bc461bcf16dccd28dc79c -->
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to