search

holmari / gerritstats

Tool for creating statistics from a Gerrit repository
MIT License
195 stars 54 forks source link

README mentions option "list-commits-exceeding-patch-set-count" for gerrit_stats.sh but it does not seem to exist #79

Open SylvainDe opened 10 months ago

SylvainDe commented 10 months ago

As per the title, it looks to me like the option mentionned in the README does not exist anymore.

Also, using "git log -p -S list-commits-exceeding-patch-set-count", I stumbled upon a related commit https://github.com/holmari/gerritstats/commit/b9a5ba74eb4900ce1984588ea58661e724b5b1c1 :

commit b9a5ba74eb4900ce1984588ea58661e724b5b1c1
Author: Lasse Holmstedt <lasse.holmstedt@gmail.com>
Date:   Tue Aug 25 22:03:44 2015 +0200

    Create a chart of commits that exceed n patch sets

    Rename the command line argument to --commit-patch-set-count-threshold in
    preparation to always make the listing occur, with some default value set.

But even trying to use "commit-patch-set-count-threshold" did not quite work for me and it looks like this was deleted with https://github.com/holmari/gerritstats/commit/1b08f250233eaa0d2c327a352961ac665633fa02 :

commit 1b08f250233eaa0d2c327a352961ac665633fa02
Author: Lasse Holmstedt <lasse.holmstedt@gmail.com>
Date:   Tue Sep 27 22:27:16 2016 -0700

    Add pmd to gradle check step

    The config file is from
    https://github.com/GoogleCloudPlatform/google-cloud-java
    with minor modifications, and licensed under Apache license.

    As with checkstyle, this can be run with:
    ./gradlew check
    ./gradlew clean check

I'm happy to submit a PR to update the doc if needed.