srdo commented on a change in pull request #3154: MINOR: update RELEASING doc 
to include more details
URL: https://github.com/apache/storm/pull/3154#discussion_r339263055
 
 

 ##########
 File path: RELEASING.md
 ##########
 @@ -56,3 +83,5 @@ Ensure you have a signed GPG key, and that the GPG key is 
listed in the Storm KE
 1. Go to http://repository.apache.org and drop the staging repository.
 
 2. Delete the staged distribution files from 
https://dist.apache.org/repos/dist/dev/storm/
+
+3. Delete the git tag.
 
 Review comment:
   I don't think we should delete the git tag. Keeping it around makes it 
easier for people to find the source in git later, without having to go 
download the source zip (which would then not have any history). 
   
   I think keeping it around is/has been standard procedure on this and other 
Apache projects

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to