janhoy commented on code in PR #4842:
URL: https://github.com/apache/solr/pull/4842#discussion_r3903684858


##########
changelog/unreleased/PR#4842-update-all-non-major-dependencies.yml:
##########
@@ -0,0 +1,7 @@
+title: Update all non-major dependencies

Review Comment:
   These auto generated changelog files are not very precise for grouped 
updates.
   
   So either skip changelog for this, or edit the title with a comma separated 
list of all deps upgraded.
   
   And realistically we will already have changelog files in `unreleased` 
folder touching many of the same libs, so there will be duplicates. Perhaps 
there is no avoiding the RM having to de-duplicate by hand (or LLM) before a 
release, so that each dependency bump is mentioned only once and with the 
actual latest version.
   
   And David wants no `dependency_update` entries at all, while I believe they 
are important, although perhaps noisy and a bit annoying to keep up to date. 
Will need to discuss this more elsewhere...



-- 
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