[GitHub] [arrow] thisisnic commented on a change in pull request #10774: ARROW-12701: [Website][Release] Include Rust and DataFusion commits, contributors, changes in release note

[GitBox <gi...@apache.org>]

thisisnic commented on a change in pull request #10774:
URL: https://github.com/apache/arrow/pull/10774#discussion_r675692005



##########
File path: dev/release/post-03-website.sh
##########
@@ -115,7 +140,6 @@ This is a major release covering more than ${rough_n_development_months} months
 This release includes ${n_commits} commits from ${n_contributors} distinct contributors.
 
 \`\`\`console
-$ ${contributors_command_line}

Review comment:
       Originally there were two variables:
   
   1. the code to run to get the list of commuters and number of commits
   2. the output of running the code from 1
   
   I think it was done like that before so that the variable 1 could be displayed in the output.  However, now we use longer commands to generate the list of committers/commits, it's no longer desirable to display it and so this has been simplified by just using a single variable.




-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org