ianmcook commented on a change in pull request #10774:
URL: https://github.com/apache/arrow/pull/10774#discussion_r675771356
##########
File path: dev/release/post-03-website.sh
##########
@@ -55,14 +57,38 @@ rough_n_development_months=$((
git_tag=apache-arrow-${version}
git_range=apache-arrow-${previous_version}..${git_tag}
-committers_command_line="git shortlog -csn ${git_range}"
-contributors_command_line="git shortlog -sn ${git_range}"
-
-committers=$(${committers_command_line})
-contributors=$(${contributors_command_line})
+# DataFusion directory and git ranges should be added here once git tagging
convention confimed with DF team
Review comment:
Add a comment here to provide context:
```suggestion
# Previously this code counted commits, committers, and contributors only to
the
# apache/arrow repository. After the Rust implementation of Arrow was moved
to
# the separate apache/arrow-rs repository, this code was modified to also
count
# commits, committers, and contributors to that repository.
#
# TODO: Add DataFusion directory and git ranges here once git tagging
convention
# for apache/arrow-datafusion repo is confirmed with DF team
#
# TODO: Consider counting other apache/arrow-* repos here as tagging
conventions
# permit (arrow-cookbook, arrow-site)
```
--
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]
