Github user knusbaum commented on the issue:

    https://github.com/apache/storm/pull/1468
  
    -1
    I am generally opposed to this. Most PRs only have a small number of 
commits and aren't a problem. For PRs with a large number of commits, it's 
simple enough to ask the contributor to squash their own commits. 
    
    I'm not sure I see the benefit in adding another script, which we will have 
to maintain, in order to do something we should rarely be doing. 
    
    Also, I worry that having this script will lead to a sharp rise in 
totally-squashed PR merges, even when there's not really any benefit (and in 
fact, loss of authorship info) since some people are likely just going to use 
the script whenever they're doing a merge.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to