I’d like to have both committers and contributers included, not just contributors. If it’s just contributors, and the list is short, then it may appear we have a very small community ;-).
I do like the idea of thanking folks! > On Apr 26, 2024, at 9:45 PM, Chris Hostetter <hossman_luc...@fucit.org> wrote: > > > : LOL meanwhile I posted https://github.com/apache/solr/pull/2424 for > : the script I developed and improved today. > : I think CHANGES.txt is the best source for a release centric view > : while git log is best for project health metrics. > > Agreed. People are frequently mentioned in CHANGES because they > contributed to the *issue* even when they didn't contribute to the *code* > (ie: reported & diagnosed a bug, aided in design discussions, etc..) > > A script to use git commit data to help create CHANGES entries (or look > for CHANGES entries that are missing credit) seems like a good sanity > check to ensuring nothing trivial is overlooked in CHANGES. > > A script to generate a thank you list of contributors should be based on > the list of contributors from CHANGES (regardless of how they got there) > > > > : On Fri, Apr 26, 2024 at 4:38 PM Jan Høydahl <jan....@cominvent.com> wrote: > : > > : > I think it is a good idea to include a list of contributors in the > release note mail. > : > it is a tiny encouragement for folks to contribute more. The list should > perhaps > : > be excluding committers, so we only list external contributors? > : > > : > I already added a script to dev-tools to parse SolrBot contributions from > git log and add to CHANGES: > : > > https://github.com/apache/solr/blob/main/dev-tools/scripts/addDepsToChanges.py > : > > : > Based on this I did a similar script that parses out Authors and > Co-Authored-By from git log > : > since last release, see https://github.com/apache/solr/pull/2423 for a > Draft. > : > > : > There's a risk of this method missing the names of some contributors who > did not actually commit anything to a PR but still are listed in CHANGES for > the release. That can be fixed by us being more careful when merging PRs, and > when committing patches from JIRA, > : > > : > Jan > : > > : > > 26. apr. 2024 kl. 15:39 skrev David Smiley <dsmi...@apache.org>: > : > > > : > > On Fri, Apr 26, 2024 at 9:35 AM Gus Heck <gus.h...@gmail.com> wrote: > : > >> > : > >> I don't know if it's relevant, but I recall that back in the early > 2000's > : > >> around the time of the adoption of the ASL 2.0 (when I was > contributing to > : > >> Ant) the ASF had us stop using @author tags in code. I was not a fan > at the > : > >> time, but they had some reason I don't fully recall relating to > shielding > : > >> the contributors in the event of someone hitting a bug and then trying > to > : > >> sue folks to recover losses or something. I wonder if that logic still > : > >> exists, and if this could be seen as related to that. It's also > possible > : > >> that this memory has severely mutated while hanging out in the back of > my > : > >> brain for 20 year :). > : > > > : > > The context of the name appearing as I propose in a "thank you" is > : > > merely to thank them, not to indirectly hold them to stability/quality > : > > measures. > : > > > : > > I don't think it's related. @author tags can repel a collaborative > : > > ownership mindset on a specific bit of code. I used to @author my > : > > code out of pride but long ago I realized those tags are a bad idea > : > > and also kind of needless with git-blame anyway. > : > > > : > > --------------------------------------------------------------------- > : > > To unsubscribe, e-mail: dev-unsubscr...@solr.apache.org > : > > For additional commands, e-mail: dev-h...@solr.apache.org > : > > > : > > : > : --------------------------------------------------------------------- > : To unsubscribe, e-mail: dev-unsubscr...@solr.apache.org > : For additional commands, e-mail: dev-h...@solr.apache.org > : > : > > -Hoss > http://www.lucidworks.com/ > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@solr.apache.org > For additional commands, e-mail: dev-h...@solr.apache.org _______________________ Eric Pugh | Founder | OpenSource Connections, LLC | 434.466.1467 | http://www.opensourceconnections.com <http://www.opensourceconnections.com/> | My Free/Busy <http://tinyurl.com/eric-cal> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed <https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw> This e-mail and all contents, including attachments, is considered to be Company Confidential unless explicitly stated otherwise, regardless of whether attachments are marked as such.