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 actual code to run to get the list of committers 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to