Github user spmallette commented on the issue:

    https://github.com/apache/incubator-tinkerpop/pull/333
  
    This looks much better to me - thanks. If you could just clean up the git 
history now a little bit as we've had a lot of back and forth to get this 
right. Let's squash all of your commits into one:
    
    ```text
    git rebase -i tp31-graphml
    ```
    
    You should get a summary of all your commits listed. The instructions 
explain what to do, but basically, just change all but the first "pick" entry 
to "squash" and all of those ones you mark with "squash" will fold into the 
remaining "pick". so it looks like you have just once nice commit. After you do 
that, you can adjust the commit message for the squash to have just one good 
message that describes your change.  You might want to review a tutorial or two 
on `squash` before you pull the trigger if you aren't familiar:
    
    http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html
    
    Then you push:
    
    ```text
    git push your-remote tp31-graphml --force
    ```
    
    Note that "your-remote" should be the name of the git remote for your repo 
which i assume is "origin" and further note, that you need `--force` because 
you are re-writing history on that branch (i.e. 7 commits becoming 1). Once you 
do that, i would think the PR should auto-update and travis will fire off 
again. Should be good from there.
    



---
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