> 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.
That's an interesting idea. Not to add CHANGES entries for the smallest ref-guide typo. Most PRs that link to a JIRA should be significant enough to have a CHANGES, so if we enumerate all commits with JIRA links, and all CHANGES entries wil JIRA link, then highlighting the diff would help the RM complete CHANGES and also find missing contributors. I like pet projects like this, perhaps I find some spare time. Jan > 27. apr. 2024 kl. 03:45 skrev Chris Hostetter <[email protected]>: > > > : 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 <[email protected]> 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 <[email protected]>: > : > > > : > > On Fri, Apr 26, 2024 at 9:35 AM Gus Heck <[email protected]> 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: [email protected] > : > > For additional commands, e-mail: [email protected] > : > > > : > > : > : --------------------------------------------------------------------- > : To unsubscribe, e-mail: [email protected] > : For additional commands, e-mail: [email protected] > : > : > > -Hoss > http://www.lucidworks.com/ > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
